pegthing

pegthing copied to clipboard

Boards with 1 and 2 rows never have a valid move. Boards with 3 rows with the wrong initial peg remove, also do not have a valid move. This change...

dancrumb

Does not work out of the box with Clojure 1.10.1

2

In `project.clj` I updated ` :dependencies [[org.clojure/clojure "1.10.1"]]`. Furthermore I add to change ``` (ns pegthing.core (:gen-class)) ``` since set is loaded with clojure.core now.

lccambiaghi

I believe this is because you're using `require` not `:require`, not that the code is even necessary anymore with set being built into clojure.core.

ConnorORourke

error when number of rows bigger than 6

2

Here is how it goes in the repl: ``` Get ready to play peg thing! How many rows? [5] 7 Here's your board: a0 b0 c0 d0 e0 f0 g0...

kindlychung

bug in valid-moves

1

There is a bug here: https://github.com/flyingmachine/pegthing/blob/master/src/pegthing/core.clj#L98-L100 Should be: ``` (defn valid-moves "Return a map of all valid moves for pos, where the key is the destination and the value is...

kindlychung

I've been following along with the book _Clojure for the Brave and True_ and I've noticed a few issues. In the book examples, `(require)` is used, when it should be...

NateChat42

Some suggestions

1

Hello Daniel, as you said "Pull requests [...] are very much welcome!", here they come ... 1) the "clojure.set" library is not used anywhere, 2) the visual representation of the...

danielgerigk

This doesn't look too great on windows

1

 I'm sure I can find ways around this but it's a bit annoying :-/

Dagur