-
Notifications
You must be signed in to change notification settings - Fork 46
Fixup cabal repl Simplicity #297
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
Conversation
@apoelstra I don't really know a good way of running |
Might be time for some github CI? I can take a look, what kind of test jobs should be run? |
@apoelstra has in own CI in NixOS, whose details we should probably merge into this project itself. |
My local CI is pretty heavy. I'm not sure it's reasonable to merge into this project. But if you want it it's here: https://github.com/apoelstra/local-nix-ci/blob/main/BlockstreamResearch.Simplicity.check-pr.nix |
Thanks I’ll take a look! |
09cf88b
to
303a49a
Compare
Recent refactoring in 1a4006c inserted txEnv.c into the wrong place. This caused `cabal repl Simplicity` to fail.
303a49a
to
89bbd57
Compare
rebased |
My local CI does not run |
Just to make sure I didn't somehow break something else. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 89bbd57; successfully ran local tests
Recent refactoring in 1a4006c inserted txEnv.c into the wrong place. This caused
cabal repl Simplicity
to fail.