Skip to content
This repository was archived by the owner on Feb 11, 2018. It is now read-only.

Add leap exercise - #1

Open
mbertheau wants to merge 1 commit into
masterfrom
leap
Open

Add leap exercise#1
mbertheau wants to merge 1 commit into
masterfrom
leap

Conversation

@mbertheau

Copy link
Copy Markdown
Owner

No description provided.

@lexi-lambda

Copy link
Copy Markdown

LGTM, though instead of defining your own divisible? function, you could use divides? from math/number-theory. Just note that the argument order is swapped, however. Also, I think I'd name is-leap-year? just leap-year? instead, but that's a minor note.

@mbertheau

Copy link
Copy Markdown
Owner Author

Thanks, I made all changes you suggested.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants