Skip to content

due to an out of date version of clj-tuple being included via the now…#11

Open
comprehendreality wants to merge 1 commit into
Factual:masterfrom
clojure-society:master
Open

due to an out of date version of clj-tuple being included via the now…#11
comprehendreality wants to merge 1 commit into
Factual:masterfrom
clojure-society:master

Conversation

@comprehendreality
Copy link
Copy Markdown

… out of date byte-streams dependency, clj-leveldb will fail at runtime when being required on clojure 1.9+. This is because of a newly enforced spec conform on the defn- function which the out of date clj-tuple is calling incorrectly.

excluding the old clj-tuple and including the latest addresses the issue.

… out of date byte-streams dependency, clj-leveldb will fail at runtime when being required on clojure 1.9+. This is because of a newly enforced spec conform on the defn- function which the out of date clj-tuple is calling incorrectly.

  excluding the old clj-tuple and including the latest addresses the issue.
@comprehendreality
Copy link
Copy Markdown
Author

lein test succeeds.

instead of manually pulling in clj-tuple, i tried to pull in a newer byte-streams first, but that caused the tests to fail when trying to assert that keys deleted from the db are equal to nil.

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.

1 participant