Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

fix: Clean up Log Directive hex output #97

Merged
merged 2 commits into from
Feb 16, 2023
Merged

Conversation

vezenovm
Copy link
Contributor

Related issue(s)

Resolves #94

Summary of changes

We just now trim the zeroes for hex output.

Dependency additions / changes

(If applicable.)

Test additions / changes

I have seen the result of this output when testing the println PR on the Noir repo.

Checklist

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt with default settings.
  • I have linked this PR to the issue(s) that it resolves.
  • I have reviewed the changes on GitHub, line by line.
  • I have ensured all changes are covered in the description.

Additional context

(If applicable.)

acvm/src/pwg/directives.rs Outdated Show resolved Hide resolved
@vezenovm vezenovm merged commit d23c735 into master Feb 16, 2023
@github-actions github-actions bot mentioned this pull request Feb 16, 2023
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.

Have Directive::Log output format match println eval in acir gen
3 participants