Skip to content
This repository was archived by the owner on Dec 2, 2024. It is now read-only.

Move marconi executable to the right directory#702

Merged
berewt merged 2 commits intomainfrom
marconi-nix
Sep 9, 2022
Merged

Move marconi executable to the right directory#702
berewt merged 2 commits intomainfrom
marconi-nix

Conversation

@berewt
Copy link
Contributor

@berewt berewt commented Sep 9, 2022

We must update nix now that marconi has its own directory

Pre-submit checklist:

  • Branch
    • Tests are provided (if possible)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
    • Formatting, PNG optimization, etc. are updated
  • PR
    • Self-reviewed the diff
    • Useful pull request description
    • Reference the ADR in the PR and reference the PR in the ADR (if revelant)
    • Reviewer requested

@berewt berewt requested a review from raduom September 9, 2022 15:05
@raduom raduom requested a review from zeme-iohk September 9, 2022 15:18
Copy link
Contributor

@raduom raduom left a comment

Choose a reason for hiding this comment

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

LGTM.

I would also like to have nix build github:input-output-hk/plutus-apps#marconi work, but it's already better than what we had.

@berewt berewt merged commit 8a4ebeb into main Sep 9, 2022
@berewt berewt deleted the marconi-nix branch September 9, 2022 15:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants