ExHammer / hammer

An Elixir rate-limiter with pluggable backends
https://hexdocs.pm/hammer/
MIT License
749 stars 42 forks source link

Upgrade dev dependencies: credo, ex_doc, dialyxir #61

Closed JuneKelly closed 1 year ago

JuneKelly commented 1 year ago

Upgrade:

This will close some of those pesky dependabot issues.

JuneKelly commented 1 year ago

Incidentally, I think we have some (caching?) issues in CI:

image

This kind of failure was fixed in #60, and the tests all pass locally. I'll look into this.

EDIT: I now wonder if the tests still fail sporadically in one of the environments in the test matrix, which I just now realised exists.

epinault commented 1 year ago

Hm.. strange . 22.04 and latest elixir seems strange to fail

epinault commented 1 year ago

maybe we just punt on 22.? because seems to be that 23 and 24 have been there long enough at this point

JuneKelly commented 1 year ago

Sorry it's taken a while to get back to this, I've been busy 😅

Sure, I agree, let's just test on 23 and 24, if we can't figure this out easily.