Skip to content

docs: fix parameter name inconsistency in profiler usage#7938

Closed
0xbryer wants to merge 1 commit intonoir-lang:masterfrom
0xbryer:patch-1
Closed

docs: fix parameter name inconsistency in profiler usage#7938
0xbryer wants to merge 1 commit intonoir-lang:masterfrom
0xbryer:patch-1

Conversation

@0xbryer
Copy link

@0xbryer 0xbryer commented Apr 5, 2025

Description

I noticed that the parameter --artifact-name was used in the profiler command, but based on other parts of the documentation, it seems like the correct name should be --artifact-path.

Additional Context

This change ensures consistency throughout the guide and aligns with the other examples that use --artifact-path.

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 and/or cargo fmt on default settings.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 5, 2025

Thank you for your contribution to the Noir language.

Please do not force push to this branch after the Noir team have started review of this PR. Doing so will only delay us merging your PR as we will need to start the review process from scratch.

Thanks for your understanding.

@TomAFrench TomAFrench mentioned this pull request Apr 8, 2025
5 tasks
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