Skip to content

fix(docs): bar -> foo correction#8289

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

fix(docs): bar -> foo correction#8289
ciaranightingale wants to merge 1 commit intonoir-lang:masterfrom
ciaranightingale:patch-1

Conversation

@ciaranightingale
Copy link

Description

I noticed that the docs referenced bar::from_foo() when this function is from the parent module foo.nr

Let me know if I am misunderstanding!

Problem*

Comments reference function from child module rather than parent module.

Documentation*

https://noir-lang.org/docs/noir/modules_packages_crates/modules#referencing-a-parent-module

Check one:

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

I noticed that the docs referenced `bar::from_foo()` when this function is from the parent module `foo.nr` 

Let me know if I am misunderstanding!
@github-actions
Copy link
Contributor

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.

Copy link
Contributor

@aakoshh aakoshh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@aakoshh aakoshh changed the title bar -> foo correction fix(docs): bar -> foo correction Apr 30, 2025
Copy link
Contributor

@michaeljklein michaeljklein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

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.

3 participants