rail-fence-cipher: update tests to v1.1.0 #1261

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

Closed

mrcfps wants to merge 2 commits into exercism:master from mrcfps:rail-fence-cipher-1216

mrcfps commented Feb 14, 2018

Fix #1216. Besides updating test, I've modified stub file in two places:

Adjust the order of function parameters to be consistent with test file.
Remove fence_pattern function to give learners more freedom.

mrcfps added 2 commits

February 14, 2018 17:24


          rail-fence-cipher: update tests to v1.1.0

8b2229e


          rail-fence-cipher: fix stub file

eb04f9c

Author

mrcfps commented Feb 14, 2018

Test update has moved to #1262. I'll fix stub file in a new branch.

mrcfps closed this

mrcfps deleted the rail-fence-cipher-1216 branch

February 14, 2018 13:28

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

Labels

None yet