Nothing Special   »   [go: up one dir, main page]

Skip to content
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

Nix flake suppport and CI #43

Merged
merged 35 commits into from
Nov 26, 2024
Merged

Nix flake suppport and CI #43

merged 35 commits into from
Nov 26, 2024

Conversation

sertel
Copy link
Contributor
@sertel sertel commented Aug 6, 2024

Dear SSProvers,

this PR adds the following:

  • support loading SSProve via Nix flakes
  • additional CI jobs:
    • to test the added Nix flake
    • to test the CI jobs that are now also running as part of the Nix-based Coq CI (meant to ease debugging)
  • updated documentation on the installation options

(Note that all CI jobs are passing on the branch of this PR. The latest failures were due to Azure.)

sertel added 30 commits March 13, 2024 14:37
@haselwarter
Copy link
Contributor

Hi Sebastian, thank you for the PR. I'm not a nix user, but the changes look good. It would be good if the nix installation instructions were sufficiently detailed to allow a novice (such as myself) to use the nix install. This would also allow me to test the PR. As you mentioned, the CI passes, so this is not strictly necessary, but maybe you can add a bit of detail.

@sertel
Copy link
Contributor Author
sertel commented Aug 12, 2024

Hi Philipp, of course. Let me add a tiny quick start guide.

@sertel
Copy link
Contributor Author
sertel commented Sep 2, 2024

Hi Philipp,
Is there anything else that I should add to this PR?

@4ever2 4ever2 merged commit df91382 into SSProve:main Nov 26, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants