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.

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

Tracking issue: Adding noir stdlib to npm #859

Closed
kevaundray opened this issue Feb 16, 2023 · 2 comments
Closed

Tracking issue: Adding noir stdlib to npm #859

kevaundray opened this issue Feb 16, 2023 · 2 comments
Labels
enhancement New feature or request stdlib Standard library shipped with Noir tooling

Comments

@kevaundray
Copy link
Contributor

Problem

Currently to compile noir code in the browser using the stdlib, you need to bring a copy of the stdlib and add it to the dependency graph. This is what nargo is doing.

Solution

I was looking at grain and they allow you to install the grain stdlib from npm. This seems like a possible solution for us also, where we publish the stdlib on every new noir version to keep it in sync.

See here: https://github.com/grain-lang/grain/tree/main/stdlib#installation

Alternatives considered

Additional context

@kevaundray kevaundray added enhancement New feature or request stdlib Standard library shipped with Noir tooling labels Feb 16, 2023
@kevaundray
Copy link
Contributor Author

One disadvantage/deal breaker that Blaine mentioned was that it is easy to then have the compiler version and the stdlib be out of sync, since one does not need to update the npm package.

@TomAFrench
Copy link
Member

This isn't needed after #973

@TomAFrench TomAFrench closed this as not planned Won't fix, can't repro, duplicate, stale Mar 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request stdlib Standard library shipped with Noir tooling
Projects
None yet
Development

No branches or pull requests

2 participants