-
Notifications
You must be signed in to change notification settings - Fork 17
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
Update dependencies and add Nix flake #75
base: master
Are you sure you want to change the base?
Conversation
This also should close #74, though not till it's incorporated into a release |
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.
h3-java update looks good to me
In addition, I just added a little more housekeeping, to try to prepare for adding CI back in (using GH actions, maybe? The cljs branch definitely will need it, since there's more targets to check!). I added a basic Nix configuration so there's a reproducible environment that can be generated, and I added |
@willcohen great to see you are messing around with this again! I am afraid I haven't touched it in a while. RE:
Can you remind me where we left off on that? I guess we took the old CI off because travis shut down and we never got a new one set up? |
I don't think we ever discussed it! I think what you said is exactly the current state of it. GitHub Actions seems like they work fine for lots of other projects. The main blocker for me and the clojurescript branch is the fact that I have to rewrite the entire test suite since midje isn't cross compatible, so it's a good chance to rethink through CI since the test suite could see a big change soon too. |
I'm working on dusting off my branch for #69, so in advance of that, I thought it might be worth tidying up the aging dependency list on the main branch first!