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

SCP-2512: plutus-use-cases-scripts is now run during Hydra CI #3594

Merged
merged 1 commit into from
Jul 23, 2021

Conversation

koslambrou
Copy link
Contributor

plutus-use-cases-scripts is now run during Hydra CI and the result is available for download from Hydra.

Pre-submit checklist:

  • Branch
    • Commit sequence broadly makes sense
    • Key commits have useful messages
    • Relevant tickets are mentioned in commit messages
    • Formatting, materialized Nix files, PNG optimization, etc. are updated
  • PR
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

Pre-merge checklist:

  • Someone approved it
  • Commits have useful messages
  • Review clarifications made it into the code
  • History is moderately tidy; or going to squash-merge

@koslambrou koslambrou requested a review from j-mueller July 22, 2021 12:17
@koslambrou koslambrou force-pushed the scp-2512-puc-scripts-hydra branch from 22ef154 to b9718f3 Compare July 22, 2021 14:03
ln -s ${haskell.packages.plutus-use-cases.src}/scripts/protocol-parameters.json protocol-parameters.json
${puc-scripts-invoker}/bin/plutus-use-cases-scripts $out/scripts scripts
# Mainnet address is used because no networkid is specified (with the '-n' flag)
${puc-scripts-invoker}/bin/plutus-use-cases-scripts $out/transactions transactions -p protocol-parameters.json
Copy link
Contributor

Choose a reason for hiding this comment

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

can you also please make a tar or zip file containing both folders? That's easier than downloading each script individually

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh yeah, good idea!

@koslambrou koslambrou force-pushed the scp-2512-puc-scripts-hydra branch from b9718f3 to 96cc9a0 Compare July 22, 2021 14:22
@koslambrou koslambrou force-pushed the scp-2512-puc-scripts-hydra branch from 96cc9a0 to c598bd1 Compare July 22, 2021 14:41
@j-mueller
Copy link
Contributor

Yes that looks good, thanks!

@j-mueller j-mueller merged commit b8ff369 into master Jul 23, 2021
@koslambrou koslambrou deleted the scp-2512-puc-scripts-hydra branch July 23, 2021 11:16
generated-puc-scripts-output = pkgs.runCommand "plutus-use-cases-scripts-output" { } ''
mkdir -p $out/scripts
mkdir -p $out/transactions
ln -s ${haskell.packages.plutus-use-cases.src}/scripts/protocol-parameters.json protocol-parameters.json
Copy link
Contributor

Choose a reason for hiding this comment

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

You can just pick up the file from the source tree, no need to do this roundabout thing. Something like ${builtins.readFile ./scripts/protocol-paramters.json}

Copy link
Contributor

Choose a reason for hiding this comment

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

Not a big deal, though.

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