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

leap: Remove recommendation to DIY in description #778

Merged
merged 1 commit into from
Dec 17, 2018
Merged

leap: Remove recommendation to DIY in description #778

merged 1 commit into from
Dec 17, 2018

Conversation

petertseng
Copy link
Member

Leap-year functions don't generally occur in standard libraries.

Is there a single example of a language we have that does this?

If there is one, then that would be the right way to solve this exercise
in that language. Presumably this exercise is primarily placed at the
top of core tracks. We should not encourage to re-implement standard
library functions at that point. But this is a hypothetical argument.

exercism/problem-specifications#1418

Leap-year functions don't generally occur in standard libraries.

Is there a single example of a language we have that does this?

If there is one, then that would be the right way to solve this exercise
in that language. Presumably this exercise is primarily placed at the
top of core tracks. We should not encourage to re-implement standard
library functions at that point. But this is a hypothetical argument.

exercism/problem-specifications#1418
@petertseng petertseng added the sync/readme Keep a README in sync with exercism/problem-specifications label Dec 17, 2018
@petertseng
Copy link
Member Author

By the way, since all other otherwise-active maintainers declared themselves unavailable, I'm merging this without review. Post-merge review is encouraged when maintainers become available again.

@petertseng
Copy link
Member Author

and because this blocks other PRs that are to be merged.

@petertseng petertseng merged commit 854c103 into exercism:master Dec 17, 2018
@petertseng petertseng deleted the leap branch December 17, 2018 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sync/readme Keep a README in sync with exercism/problem-specifications
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants