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.

Sign up for GitHub

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

Jump to bottom

Fix crypto-square to use a square, not groups of five #22

Closed

kytrinyx opened this issue Oct 15, 2014 · 1 comment

Member

kytrinyx commented Oct 15, 2014

Somewhere along the line we lost the square part of the crypto square problem, replacing the grouping with the standard groupings of 5.

The test suite should be updated to reflect the squarishness of this problem.

See exercism/problem-specifications#28

bressain mentioned this issue

Updated the unit tests to match the spec in the Readme #25

Merged

Contributor

rprouse commented Nov 14, 2014

Fixed by PR #25

bressain closed this as completed

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