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

Some suggestions #9

Open

danielgerigk wants to merge 3 commits into flyingmachine:master from danielgerigk:master

danielgerigk commented Dec 22, 2015

Hello Daniel,

as you said "Pull requests [...] are very much welcome!", here they come ...

the "clojure.set" library is not used anywhere,
the visual representation of the board could be improved ( - but, of course, this is subject to personal taste - ),
as it stands, it is possible for the user to move a peg from position a to position b, even if there is no peg at position a.

Best,
Daniel Gerigk

danielgerigk added 3 commits

December 22, 2015 23:42


          library not required

8ae9218


          improve the look of the board

56f2c07


          test whether there is a peg at pos

28519e0

ferrucc-io commented Sep 26, 2018

I found that without removing the clojure.set library the program crashes, so this PR is needed!

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

Labels

None yet