-
Notifications
You must be signed in to change notification settings - Fork 200
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
feat: Add initial version of noir.js #2681
Conversation
Updated dependencies detected. Learn more about Socket for GitHub ↗︎
|
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
On second thoughts, might it be safer to require the version bump to have been done already manually? This makes running the release workflow idempotent. |
Not a blocker. |
yeah I don't mind doing that |
Actually, let's merge this and then we can address both workflows together. |
Oh sorry already pushed it |
* tf/split-build-and-test: chore: remove reference to `result` directory chore!: Change `noir-lang/noir-source-resolver` to `noir-lang/source-resolver` (#2718) chore: clippy fixes (#2719) chore: fix version querying chore: nits chore(ci): fix playwright caching chore: pass full `package.json` into build artifact chore: clean up straggler `packageManager` chore: remove `packageManager" fields from workspace packages chore: refresh lockfile Update .github/workflows/abi_wasm.yml chore: split build and tests jobs in `abi_wasm.yml` chore: Integrate noir.js into workspace and CI release workflow (#2705) feat: Add initial version of noir.js (#2681) chore: switch `release-source-resolver.yml` to use yarn (#2704) chore: move stranded test to correct directory (#2701) chore(ci): deny wildcard dependencies (#2702) chore: fix `noirc_abi_wasm` publish for release-please (#2699)
* master: chore: Add deprecation message for default type in for loop (#2689) chore!: Change `noir-lang/noir-source-resolver` to `noir-lang/source-resolver` (#2718) chore: clippy fixes (#2719) chore: Integrate noir.js into workspace and CI release workflow (#2705) feat: Add initial version of noir.js (#2681) chore: switch `release-source-resolver.yml` to use yarn (#2704) chore: move stranded test to correct directory (#2701) chore(ci): deny wildcard dependencies (#2702) chore: fix `noirc_abi_wasm` publish for release-please (#2699) chore: introduce workspace for js packages (#2677) feat: Allow methods defined in a contract to be non-entry points (#2687) chore: revert some unwanted small changes from PR2180 (#2685) feat(lsp): Add nargo capabilities for test metadata (#2532) chore(ci): Add `cargo deny` workflow (#2682)
* master: chore(ci): Treat clippy warnings as errors in CI (#2684) chore(ci): Cache yarn and playwright dependencies in CI (#2697) chore: Add noir-wasm publishing as a workflow (#2721) fix: failing js tests (#2722) chore: Add deprecation message for default type in for loop (#2689) chore!: Change `noir-lang/noir-source-resolver` to `noir-lang/source-resolver` (#2718) chore: clippy fixes (#2719) chore: Integrate noir.js into workspace and CI release workflow (#2705) feat: Add initial version of noir.js (#2681) chore: switch `release-source-resolver.yml` to use yarn (#2704) chore: move stranded test to correct directory (#2701) chore(ci): deny wildcard dependencies (#2702) chore: fix `noirc_abi_wasm` publish for release-please (#2699)
Description
A simple package that exports
acvm_js
andnoirc_abi
Problem*
Resolves noir-lang/Planning#31
Resolves noir-lang/Planning#32
Summary*
Creates a package that exports acvm_js and noirc_abi
Documentation
This PR requires documentation updates when merged.
Additional Context
PR Checklist*
cargo fmt
on default settings.