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

Add flake.nix #811

Merged
merged 3 commits into from
Apr 20, 2023
Merged

Add flake.nix #811

merged 3 commits into from
Apr 20, 2023

Conversation

manveru
Copy link
Contributor

@manveru manveru commented Mar 16, 2023

Content

This PR includes the flake.nix and flake.lock files, enabling easy building using nix for the mithril-client, mithril-signer, and mithril-aggregator packages.

It also adds a minimal devShell for cargo and rustc, as well as setup for formatting using the standard nix fmt or treefmt commands.

Pre-submit checklist

  • Branch
    • Tests are provided (if possible)
    • Crates versions are updated (if relevant)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • No clippy warnings in the CI
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested
  • Documentation
    • Update README file (if relevant)
    • Update documentation website (if relevant)
    • Add dev blog post (if relevant)

Comments

Once merged, they can be used with nix build or nix run under the following flake URLs:

  • github:input-output-hk/mithril#mithril-client
  • github:input-output-hk/mithril#mithril-signer
  • github:input-output-hk/mithril#mithril-aggregator

@github-actions
Copy link

github-actions bot commented Mar 16, 2023

Test Results

    3 files  ±0    30 suites  ±0   6m 19s ⏱️ +33s
470 tests ±0  470 ✔️ ±0  0 💤 ±0  0 ±0 
540 runs  ±0  540 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 4a7a11c. ± Comparison against base commit bcad6b1.

♻️ This comment has been updated with latest results.

@manveru manveru temporarily deployed to testing-preview March 16, 2023 11:12 — with GitHub Actions Inactive
@danielmain danielmain temporarily deployed to testing-preview March 17, 2023 14:41 — with GitHub Actions Inactive
@manveru manveru temporarily deployed to testing-preview March 21, 2023 07:34 — with GitHub Actions Inactive
Copy link
Member

@jpraynaud jpraynaud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@jpraynaud jpraynaud merged commit 01921eb into main Apr 20, 2023
@jpraynaud jpraynaud deleted the add-flake-nix branch April 20, 2023 16:08
@jpraynaud jpraynaud mentioned this pull request Apr 21, 2023
5 tasks
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