-
Notifications
You must be signed in to change notification settings - Fork 84
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
add support for nix, like #705 but as branch in this repo #706
Conversation
We can merge this as is, right? |
Yes, it should be good to go. I don't have the rights to rebase and update it though. |
I rebased it, if CI goes green this should be fine |
Thanks guys! |
- reopened | ||
push: | ||
branches: | ||
- master |
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.
FWIW, this value should probably be coq-8-14
so that the workflow runs when the branch is updated. But for now, the Coq Nix Toolbox generation mechanism always put master
, so you'll probably have to fix this manually (and it could be worth opening an issue on the Coq Nix Toolbox repo about this).
- reopened | ||
push: | ||
branches: | ||
- master |
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.
Ditto.
* add support for nix * hack around cachedMake script of coq-nix-toolbox * update coq-nix-toolbox version * testing with mac os CI Co-authored-by: Kenji Maillard <[email protected]>
* add support for nix * hack around cachedMake script of coq-nix-toolbox * update coq-nix-toolbox version * testing with mac os CI Co-authored-by: Kenji Maillard <[email protected]>
No description provided.