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

Generator implements InputRange interface #5309

Closed
wants to merge 1 commit into from

Conversation

dukc
Copy link
Contributor

@dukc dukc commented Mar 28, 2017

Reboot of #5194

…ement

std.range.interfaces.InputRange without having to call inputRangeObject().
@dukc
Copy link
Contributor Author

dukc commented Apr 3, 2017

ping @wilzbach (or anybody who can review)

@dukc
Copy link
Contributor Author

dukc commented Apr 12, 2017

ping @JackStouffer

@dukc
Copy link
Contributor Author

dukc commented Apr 22, 2017

Ignored, so I close. Perhaps i'll try again some day.

@dukc dukc closed this Apr 22, 2017
@JackStouffer
Copy link
Member

Sorry about this. I am currently juggling work and school so I only have time to review trivial PRs.

Please understand that if a PR is in the queue and there are no blocking issues, it will get reviewed eventually.

@dukc
Copy link
Contributor Author

dukc commented Apr 25, 2017

Well, it ain't ignored anymore, so I can reopen...

It does not matter per se, i'm not in a hurry to get this in. I just don't want it to become one of those dead pr-s littering the queue.

@dukc dukc reopened this Apr 25, 2017
@codecov-io
Copy link

Codecov Report

Merging #5309 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5309      +/-   ##
==========================================
+ Coverage   88.57%   88.57%   +<.01%     
==========================================
  Files         117      117              
  Lines       77281    77311      +30     
==========================================
+ Hits        68452    68482      +30     
  Misses       8829     8829
Impacted Files Coverage Δ
std/concurrency.d 84.27% <100%> (+0.85%) ⬆️

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 be8f1dd...f3c2e6d. Read the comment docs.

@dukc
Copy link
Contributor Author

dukc commented Jun 23, 2017

I have long since stopped checking this back often enough. If somebody made a comment on this it could well take weeks for me to notice. I close this, for now.

@dukc dukc closed this Jun 23, 2017
@MetaLang
Copy link
Member

MetaLang commented Jun 23, 2017

@dukc if you still want to go ahead with this, I don't have merge privileges but I will review it and help push it through. If not, I will re-submit it on your behalf.

@quickfur
Copy link
Member

@MetaLang I agree this should be resubmitted. I think it's worthwhile pursuing; I feel sorry that nobody had the time to review it. If only I had more free time...

@dukc
Copy link
Contributor Author

dukc commented Jun 26, 2017

Reopened here: #5515

this time i'll notify on the forums before closing it, I promise.

@WalterBright
Copy link
Member

Please don't close good work.

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.

6 participants