Skip to content

declare hydra jobs - #239

Merged
teodanciu merged 4 commits into
masterfrom
flake
Oct 10, 2023
Merged

teodanciu merged 4 commits into
masterfrom
flake

Conversation

@dermetfan

Copy link
Copy Markdown
Contributor

Description

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • Code is formatted according to CONTRIBUTING.md
  • Self-reviewed the diff

@dermetfan
dermetfan requested a review from teodanciu October 10, 2023 11:06
Comment thread flake.nix
@@ -0,0 +1,22 @@
{
outputs = _: let
systems = [ "x86_64-linux" ];

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Currently only builds for linux. Add more here if desired.

@WhatisRT WhatisRT left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you!

One extra thing that would be very nice to have would be if we could download the PDF that's contained in *.docs, but it doesn't need to be part of this PR. This was something you could do in the cardano-ledger repo in the past (by navigating hydra), and was very convenient.

@teodanciu teodanciu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me, thank you!

@teodanciu
teodanciu merged commit b287172 into master Oct 10, 2023
@teodanciu
teodanciu deleted the flake branch October 10, 2023 15:19
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