Skip to content

leap: Remove recommendation to DIY in description#778

Merged
petertseng merged 1 commit intoexercism:masterfrom
petertseng:leap
Dec 17, 2018
Merged

leap: Remove recommendation to DIY in description#778
petertseng merged 1 commit intoexercism:masterfrom
petertseng:leap

Conversation

@petertseng
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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