-
Notifications
You must be signed in to change notification settings - Fork 4
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
Wish: Composability with other actions #51
Comments
Hi @JasonGross, thanks for opening this. On the one hand, it's true that unlike "lighter" TypeScript actions (very often named setup-*) which just intend to install some new binaries directly in the GHA worker path, Container actions are a bit more "monolithic" in some sense (in particular from the point of view of docker-coq-action, the bash script intended to call But this appeared to be flexible enough to setup CI for all coq-community projects involving coq and/or ocaml. In particular, the default configuration only requires to specify the On the other hand (and you are the first user to request this), one might want to "compose" several actions such as Three related remarks:
|
Thanks for writing such a quick and detailed answer @erikmd! |
OK @Zimmi48 ! I agree the use of Nix is interesting to have more flexibility regarding the choice of a particular version of haskell (as opposed as directly installing the single version packaged in APT), but as I mentioned above, this is also directly feasible from the Docker-Coq images as well − relying on Nix :) |
It might be nice to port fiat-crypto and coq-tools to the docker coq action.
Currently fiat-crypto uses actions/setup-haskell@v1 with ghc 8.8.1 and 3.0 (and uses my ppa for Coq versions 8.12.0, 8.11.1, 8.10.2, 8.9.1, as well as the tips of master, v8.12, v8.11, v8.10, v8.9), and coq-tools uses actions/setup-python@v1 to test python versions 2.7, 3.5, 3.6, 3.7, 3.8 with Coq versions 8.11.1, 8.10.2, 8.9.1, 8.8.2, 8.7.2, 8.6.1, 8.5pl3, and the tips of the master, v8.11, v8.10, v8.9, v8.8, v8.7, v8.6, and v8.5 branches. However, I don't think docker containers are easily composable with other actions.
The text was updated successfully, but these errors were encountered: