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

feat: print error location with fmt #497

Merged
merged 1 commit into from
Aug 18, 2023
Merged

feat: print error location with fmt #497

merged 1 commit into from
Aug 18, 2023

Conversation

sirasistant
Copy link
Contributor

Description

Problem*

Updates the string representation of opcode locations with fmt for easy parsing.

Summary*

Additional Context

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

@kevaundray kevaundray added this pull request to the merge queue Aug 18, 2023
Merged via the queue into master with commit 575a9e5 Aug 18, 2023
@kevaundray kevaundray mentioned this pull request Aug 18, 2023
Merged
@kevaundray kevaundray deleted the arv/error_print branch August 18, 2023 10:52
TomAFrench added a commit that referenced this pull request Aug 18, 2023
@TomAFrench
Merge branch 'master' into move-wasm-into-acvm
c39fe6d 
* master:
  feat(acvm_js): Add `execute_circuit_with_black_box_solver` to prevent reinitialization of `BlackBoxFunctionSolver` (#495)
  chore: Release 0.22.0 (#470)
  feat: print error location with fmt (#497)
  feat!: Switched from OpcodeLabel to OpcodeLocation and ErrorLocation (#493)
Sakapoi pushed a commit to Sakapoi/acvm_fork that referenced this pull request Aug 18, 2023
@sirasistant @Sakapoi
feat: print error location with fmt (noir-lang#497)
e20fc25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kevaundray kevaundray kevaundray approved these changes

@TomAFrench TomAFrench Awaiting requested review from TomAFrench

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sirasistant @kevaundray