Skip to content

Added two exercises#20

Merged
kytrinyx merged 2 commits intoexercism:masterfrom
mikecoop:master
Oct 9, 2014
Merged

Added two exercises#20
kytrinyx merged 2 commits intoexercism:masterfrom
mikecoop:master

Conversation

@mikecoop
Copy link
Copy Markdown
Contributor

@mikecoop mikecoop commented Oct 8, 2014

Added the difference-of-squares and secret-handshake exercises.

I didn't add them to the config file because I don't know where in the order they should go since they are both fairly basic exercises.

Added the difference-of-squares and secret-handshake exercises
@kytrinyx
Copy link
Copy Markdown
Member

kytrinyx commented Oct 8, 2014

Cool!

The build is failing with the following output:

$ bin/configlet .
Evaluating .
-> config.json does not include [secret-handshake difference-of-squares].
The command "bin/configlet ." exited with 1.

Would you add these problems to the config.json in the root of the repository?

@mikecoop
Copy link
Copy Markdown
Contributor Author

mikecoop commented Oct 9, 2014

Does the order of the exercises matter? Should more simple ones be at the beginning or just put them at the end?

@kytrinyx
Copy link
Copy Markdown
Member

kytrinyx commented Oct 9, 2014

Simpler ones should be toward the beginning... ish. It's not an exact science, as you can imagine.

@jwood803
Copy link
Copy Markdown
Contributor

jwood803 commented Oct 9, 2014

I don't see why these can't be added to the end. If I recall, that's what I've done. The difference-of-squares exercise could go more toward the beginning, but that's just my opinion.

@kytrinyx
Copy link
Copy Markdown
Member

kytrinyx commented Oct 9, 2014

Ok, I have no opinion as to what is easier or harder than anything else. If there's nothing obvious, then just stick it on the end.

@bressain
Copy link
Copy Markdown
Contributor

bressain commented Oct 9, 2014

I've just been going in the order of issue #2 which would be to tack them onto the end:

difference-of-squares,
secret-handshake

kytrinyx added a commit that referenced this pull request Oct 9, 2014
@kytrinyx kytrinyx merged commit e0210da into exercism:master Oct 9, 2014
@kytrinyx
Copy link
Copy Markdown
Member

kytrinyx commented Oct 9, 2014

Thank you!

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.

4 participants