-
Notifications
You must be signed in to change notification settings - Fork 3
Add GHC 9.10.2 & update Prometheus dependency #57
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.
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
base: master
Are you sure you want to change the base?
Conversation
Capture changes due to new version of haskell-ci.
prometheus 2.3.0 changes the behaviour of the `observeAndSample` function: - bitnomial/prometheus#51 This does not seem to require any changes in Ridley. Change log: - https://github.com/bitnomial/prometheus/blob/master/CHANGELOG.md#230
| shelly >= 1.8.2, | ||
| safe-exceptions < 1.8, | ||
| transformers, | ||
| prometheus > 0.5.0 && < 2.3.0, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @PiDelport ! I remember adding that constraint because there were a few breaking changes in prometheus-2.3.0, but I don't remember on top of my head -- have you noticed anything out of the ordinary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, i did not look very deeply: i just ran the tests and checked that nothing refers directly to observeAndSample.
|
I pushed a fixed for the build issue with |
|
A few more cleanups, and the build is green! ✅ |
This adds GHC 9.10.2 to
tested-withand the CI matrix, and updates theprometheusdependency to allow 2.3.0.Are there other GHC versions we also want to add, while here?