Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace generators with a deducer #5

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

replace generators with a deducer #5

wants to merge 11 commits into from

Conversation

arlor
Copy link

@arlor arlor commented Jan 2, 2018

replace various generators with one deducer

@thautwarm
Copy link
Member

I'm sorry for my offline these days(I'm busy with my final exams).
Thanks again for your PR, however, check it to pass CI, please.
Your Bug Here

@coveralls
Copy link

coveralls commented Jan 23, 2018

Coverage Status

Coverage decreased (-0.06%) to 93.333% when pulling e237ecd on Athedorer:master into 55e8ed3 on Xython:master.

@codecov-io
Copy link

codecov-io commented Jan 23, 2018

Codecov Report

Merging #5 into master will decrease coverage by 0.18%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #5      +/-   ##
=========================================
- Coverage   88.48%   88.3%   -0.19%     
=========================================
  Files           9       9              
  Lines         330     342      +12     
  Branches       62      69       +7     
=========================================
+ Hits          292     302      +10     
- Misses         22      23       +1     
- Partials       16      17       +1
Impacted Files Coverage Δ
after/linq/standard/generator.py 100% <0%> (ø) ⬆️
after/linq/core/collections.py 67.64% <0%> (+6.77%) ⬆️

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 55e8ed3...d655e4d. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants