Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

LSP reports use dep::std as an error #1833

Closed
Tracked by #1558
phated opened this issue Jun 28, 2023 · 0 comments · Fixed by #1840
Closed
Tracked by #1558

LSP reports use dep::std as an error #1833

phated opened this issue Jun 28, 2023 · 0 comments · Fixed by #1840
Assignees
@phated
Labels
bug Something isn't working lsp Language Server Protocol

Comments

@phated
Copy link
Contributor

phated commented Jun 28, 2023

Aim

I want to use the stdlib but not have it reported as an error by the LSP

Expected Behavior

When adding use dep::std, the LSP should not mark it as an error (missing dependency).

Bug

It highlights use dep::std in red and indicates it is an error for an unknown dependency.

To Reproduce

Installation Method

None

Nargo Version

No response

Additional Context

No response

Would you like to submit a PR for this Issue?

Yes

Support Needs

No response
@phated phated added the bug Something isn't working label Jun 28, 2023
@phated phated self-assigned this Jun 28, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Noir Jun 28, 2023
@phated phated added the lsp Language Server Protocol label Jun 28, 2023
This was referenced Jun 28, 2023
Closed
Merged
@phated phated moved this from 📋 Backlog to 🏗 In progress in Noir Jun 29, 2023
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Noir Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@phated phated

Labels
bug Something isn't working lsp Language Server Protocol
Projects
Noir
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(lsp): Ensure stdlib is always added before the check_crate phase noir-lang/noir
1 participant
@phated