Remove dep keyword for importing from dependencies
#2307
Assignees
Labels
enhancement
New feature or request
Comments
|
@jfecher did you start working on this? If not, then I think we can assign it to someone else to reduce context switch and since its E-LOW |
|
@kevaundray I haven't |
github-merge-queue bot
pushed a commit
that referenced
this issue
Jun 17, 2024
# Description ## Problem\* Resolves #2307 ## Summary\* `dep::` is unnecessary to distinguish between internal and external crates, so this PR removes it. ## 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\* - [x] I have tested the changes locally. - [x] I have formatted the changes with [Prettier](https://prettier.io/) and/or `cargo fmt` on default settings. --------- Co-authored-by: Tom French <[email protected]> Co-authored-by: Tom French <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem
Importing from libraries in Noir differs from rust in that you must add a
dep::prefix to each path in the dependency. This is unnecessary since you can already tell by the crate name in the path that it is a different crate.Happy Case
The
dep::prefix is removed.Alternatives Considered
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response
The text was updated successfully, but these errors were encountered: