FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

set: Implement initialization set with sequence by SanggiHong · Pull Request #100 · go-python/gpython · GitHub

set: Implement initialization set with sequence - #100

Merged
corona10 merged 4 commits into
go-python:masterfrom
SanggiHong:set-init-with-sequence
Sep 29, 2019
Merged

set: Implement initialization set with sequence#100
corona10 merged 4 commits into
go-python:masterfrom
SanggiHong:set-init-with-sequence

Conversation

Copy link
Copy Markdown
Contributor

Implement the initialization of Set with a sequence. looked up the implementation of the List
issue #76

codecov-io commented Sep 29, 2019
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #100 into master will increase coverage by 0.14%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master     #100      +/-   ##
==========================================
+ Coverage   69.09%   69.24%   +0.14%     
==========================================
  Files          60       60              
  Lines       10675    10690      +15     
==========================================
+ Hits         7376     7402      +26     
+ Misses       2781     2767      -14     
- Partials      518      521       +3
Impacted Files Coverage Δ
py/set.go 58.18% <66.66%> (+10%) ⬆️
py/sequence.go 61.11% <86.66%> (+6.27%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8c361a8...130d715. Read the comment docs.

Comment thread py/tests/set.py
corona10 merged commit 9c36f4c into go-python:master Sep 29, 2019
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL