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

feat: Add noir_lsp_wasm as a WASI implementation of the LSP #2769

Closed
wants to merge 31 commits into from
Closed
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
7d2280f
feat: Add noir_lsp_wasm as a WASI implementation of the LSP
phated Sep 20, 2023
5836927
align rustix version with async-lsp
phated Sep 21, 2023
73fd403
upgrade rust toolchain to build WASI correctly
phated Sep 21, 2023
733d824
regen lock
phated Sep 21, 2023
23f8f49
normalize the vscode-test-web scheme
phated Sep 21, 2023
05b2af6
msrv to 1.67.0
phated Sep 21, 2023
4781f10
Merge branch 'master' into phated/lsp-wasm
TomAFrench Sep 29, 2023
abd0ec0
chore: clippy
TomAFrench Sep 29, 2023
e1c37ad
Merge branch 'master' into phated/lsp-wasm
TomAFrench Sep 29, 2023
420e25e
chore: move dependencies down into `noir_lsp_wasm`
TomAFrench Sep 29, 2023
859423e
Merge branch 'master' into phated/lsp-wasm
TomAFrench Sep 29, 2023
7d31a7a
chore: remove unused dependencies
TomAFrench Sep 29, 2023
4901267
chore: push stdio implementation upstream
TomAFrench Sep 29, 2023
627c390
Merge branch 'master' into phated/lsp-wasm
TomAFrench Sep 29, 2023
b7997e3
Merge branch 'master' into phated/lsp-wasm
TomAFrench Sep 29, 2023
43ec9dc
Merge branch 'master' into phated/lsp-wasm
TomAFrench Oct 18, 2023
c584fdd
Merge branch 'master' into phated/lsp-wasm
TomAFrench Oct 25, 2023
02f58c7
Merge branch 'master' into phated/lsp-wasm
TomAFrench Oct 31, 2023
10561d5
chore: fix `rust-version` field
TomAFrench Oct 31, 2023
b3de1bf
chore: fix merge
TomAFrench Oct 31, 2023
1a9a5b4
chore: enforce consistent version on noir_lsp_wasm
TomAFrench Oct 31, 2023
43a15b7
chore: use upstream async-lsp
TomAFrench Oct 31, 2023
c0f8f67
Merge branch 'master' into phated/lsp-wasm
TomAFrench Nov 2, 2023
7337352
Merge branch 'master' into phated/lsp-wasm
TomAFrench Nov 2, 2023
a8a6fc2
Merge branch 'master' into phated/lsp-wasm
TomAFrench Nov 14, 2023
88b44e8
Merge branch 'master' into phated/lsp-wasm
TomAFrench Nov 17, 2023
a1a4310
chore: remove unused patch
TomAFrench Nov 17, 2023
3cd3da0
Merge branch 'master' into phated/lsp-wasm
TomAFrench Jan 12, 2024
73a26a1
Merge branch 'master' into phated/lsp-wasm
TomAFrench Apr 21, 2024
389b7a0
chore: fix build
TomAFrench Apr 21, 2024
ee07c58
chore: fix CI
TomAFrench Apr 21, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/workflows/formatting.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
timeout-minutes: 30
env:
RUSTFLAGS: -Dwarnings

strategy:
fail-fast: false
matrix:
Expand All @@ -32,7 +32,7 @@ jobs:
uses: actions/checkout@v4

- name: Setup toolchain
uses: dtolnay/rust-toolchain@1.66.0
uses: dtolnay/rust-toolchain@1.67.0
TomAFrench marked this conversation as resolved.
Show resolved Hide resolved
with:
targets: ${{ matrix.target }}
components: clippy, rustfmt
Expand All @@ -53,7 +53,7 @@ jobs:
name: eslint
runs-on: ubuntu-latest
timeout-minutes: 30

steps:
- name: Checkout
uses: actions/checkout@v4
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ jobs:
echo "MACOSX_DEPLOYMENT_TARGET=$(xcrun -sdk macosx$(sw_vers -productVersion) --show-sdk-platform-version)" >> $GITHUB_ENV

- name: Setup toolchain
uses: dtolnay/rust-toolchain@1.66.0
uses: dtolnay/rust-toolchain@1.67.0
with:
targets: ${{ matrix.target }}

Expand Down Expand Up @@ -81,7 +81,7 @@ jobs:
- name: Test built artifact
if: matrix.target == 'x86_64-apple-darwin'
run: |
cp ./target/${{ matrix.target }}/release/nargo ~/.cargo/bin/
cp ./target/${{ matrix.target }}/release/nargo ~/.cargo/bin/
yarn workspace release-tests test

- name: Upload binaries to release tag
Expand Down Expand Up @@ -129,7 +129,7 @@ jobs:
ref: ${{ inputs.tag || env.GITHUB_REF }}

- name: Setup toolchain
uses: dtolnay/rust-toolchain@1.66.0
uses: dtolnay/rust-toolchain@1.67.0
with:
targets: ${{ matrix.target }}

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ jobs:
uses: actions/checkout@v4

- name: Setup toolchain
uses: dtolnay/rust-toolchain@1.66.0
uses: dtolnay/rust-toolchain@1.67.0
with:
targets: ${{ matrix.target }}

Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/wasm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ jobs:
uses: actions/checkout@v4

- name: Setup toolchain
uses: dtolnay/rust-toolchain@1.66.0
uses: dtolnay/rust-toolchain@1.67.0

- uses: Swatinem/rust-cache@v2
with:
Expand Down Expand Up @@ -143,7 +143,7 @@ jobs:
cp -r ./compiler/wasm/downloaded/nodejs ./compiler/wasm
cp -r ./compiler/wasm/downloaded/web ./compiler/wasm
yarn workspace @noir-lang/source-resolver build

- name: Run node tests
run: yarn workspace @noir-lang/noir_wasm test:node

Expand Down
2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ version = "0.12.0"
# x-release-please-end
authors = ["The Noir Team <[email protected]>"]
edition = "2021"
rust-version = "1.66"
rust-version = "1.67"
license = "MIT OR Apache-2.0"

[workspace.dependencies]
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ Concretely the following items are on the road map:

## Minimum Rust version

This crate's minimum supported rustc version is 1.66.0.
This crate's minimum supported rustc version is 1.67.0.

## Working on this project

Expand Down
2 changes: 1 addition & 1 deletion tooling/backend_interface/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ description = "The definition of the backend CLI interface which Nargo uses for
version = "0.11.0"
authors.workspace = true
edition.workspace = true
rust-version = "1.66"
rust-version = "1.67"
license.workspace = true

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
Expand Down
4 changes: 2 additions & 2 deletions tooling/nargo/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ use rustc_version::{version, Version};

fn check_rustc_version() {
assert!(
version().unwrap() >= Version::parse("1.66.0").unwrap(),
"The minimal supported rustc version is 1.66.0."
version().unwrap() >= Version::parse("1.67.0").unwrap(),
"The minimal supported rustc version is 1.67.0."
);
}

Expand Down
6 changes: 3 additions & 3 deletions tooling/nargo_cli/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ use std::{env, fs};

fn check_rustc_version() {
assert!(
version().unwrap() >= Version::parse("1.66.0").unwrap(),
"The minimal supported rustc version is 1.66.0."
version().unwrap() >= Version::parse("1.67.0").unwrap(),
"The minimal supported rustc version is 1.67.0."
);
}

Expand Down Expand Up @@ -105,7 +105,7 @@ fn generate_compile_success_empty_tests(test_file: &mut File, test_data_dir: &Pa
r#"
#[test]
fn compile_success_empty_{test_name}() {{

// We use a mocked backend for this test as we do not rely on the returned circuit size
// but we must call a backend as part of querying the number of opcodes in the circuit.

Expand Down