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

fix: Use actual reference report #2

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

vezenovm
Copy link
Owner

This PR is a followup to noir-lang#3. noir-lang#3 added a fix and also fixed the corrupt report on master. This PR goes back to using the actual reference report that was corrupted on master.

@vezenovm
Copy link
Owner Author

We can merge this into the parent as we are referencing the commit on Noir anyway

@vezenovm vezenovm merged commit acf1279 into mv/fix-program-diff-computation Apr 23, 2024
github-merge-queue bot pushed a commit to noir-lang/noir that referenced this pull request Apr 23, 2024
)

# Description

## Problem\*

Further context can be found in
#4878

## Summary\*

This PR updates the commit to reference
vezenovm/noir-gates-diff#2 which actually checks
the report on master. We need the preceding PR in order to fix the
reference report to use the actual package names in `execution_success`.

## Additional Context



## Documentation\*

Check one:
- [ ] No documentation needed.
- [ ] Documentation included in this PR.
- [ ] **[For Experimental Features]** Documentation to be submitted in a
separate PR.

# PR Checklist\*

- [ ] I have tested the changes locally.
- [ ] I have formatted the changes with [Prettier](https://prettier.io/)
and/or `cargo fmt` on default settings.
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.

1 participant