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

ci: remove unmaintained actions, use cargo/rustup directly, add MSRV … #632

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
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
80 changes: 51 additions & 29 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,31 +36,22 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
override: true
- name: Install latest Rust toolchain
run: rustup update --no-self-update
- uses: Swatinem/rust-cache@v1
- uses: actions-rs/cargo@v1
name: Test Taplo
with:
command: test
args: -p taplo
- name: Test Taplo
run: |
cargo test -p taplo

check_wasm32:
name: Check on WASM
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
target: wasm32-unknown-unknown
toolchain: stable
override: true
- name: Install latest Rust toolchain
run: |
rustup update --no-self-update
rustup target add wasm32-unknown-unknown
- uses: Swatinem/rust-cache@v1
- working-directory: crates/taplo-wasm
run: cargo check --target wasm32-unknown-unknown
Expand All @@ -70,22 +61,53 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
override: true
- name: Install latest Rust toolchain
run: |
rustup update --no-self-update
- uses: Swatinem/rust-cache@v1
- name: Retrieve toml-test
run: |
wget https://github.com/BurntSushi/toml-test/releases/download/v1.1.0/toml-test-v1.1.0-linux-amd64.gz
gunzip toml-test-v1.1.0-linux-amd64.gz
chmod +x toml-test-v1.1.0-linux-amd64
- uses: actions-rs/cargo@v1
name: Build taplo-cli
with:
command: build
args: --bin taplo --no-default-features --features "rustls-tls,toml-test"
- name: Build taplo-cli
run: |
cargo build --bin taplo --no-default-features --features "rustls-tls,toml-test"
- name: Run toml-test
run: ./toml-test-v1.1.0-linux-amd64 ./target/debug/taplo -- toml-test

test-msrv-lib:
name: Test libraries with MSRV
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Install latest Rust toolchain
run: |
rustup update --no-self-update
rustup default 1.67
- uses: Swatinem/rust-cache@v1
- name: Test crates
run: |
cargo +1.67 check -p lsp-async-stub -p taplo-common -p taplo-lsp -p taplo
cargo +1.67 test -p lsp-async-stub -p taplo-common -p taplo-lsp -p taplo

test-msrv-bin:
name: Test binaries with MSRV
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Install latest Rust toolchain
run: |
rustup update --no-self-update
rustup default 1.74
rustup target install wasm32-unknown-unknown
- uses: Swatinem/rust-cache@v1
- name: Test taplo-cli
run: |
cargo +1.74 check -p taplo-cli
cargo +1.74 test -p taplo-cli
- name: Test taplo-wasm
working-directory: crates/taplo-wasm
run: |
cargo +1.74 check

83 changes: 29 additions & 54 deletions .github/workflows/releases.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,11 +46,8 @@ jobs:
if: startsWith(github.ref, 'refs/tags/release-taplo-0')
steps:
- uses: actions/checkout@v4
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
- name: Install latest Rust toolchain
run: rustup update --no-self-update
- uses: Swatinem/rust-cache@v1
- name: Publish to Crates.io
uses: nick-fields/retry@v2
Expand All @@ -69,11 +66,8 @@ jobs:
if: startsWith(github.ref, 'refs/tags/release-taplo-cli-0')
steps:
- uses: actions/checkout@v4
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
- name: Install latest Rust toolchain
run: rustup update --no-self-update
- uses: Swatinem/rust-cache@v1
- name: Publish to Crates.io
uses: nick-fields/retry@v2
Expand All @@ -92,11 +86,8 @@ jobs:
if: startsWith(github.ref, 'refs/tags/release-taplo-common-0')
steps:
- uses: actions/checkout@v4
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
- name: Install latest Rust toolchain
run: rustup update --no-self-update
- uses: Swatinem/rust-cache@v1
- name: Publish to Crates.io
uses: nick-fields/retry@v2
Expand All @@ -115,11 +106,8 @@ jobs:
if: startsWith(github.ref, 'refs/tags/release-taplo-lsp-0')
steps:
- uses: actions/checkout@v4
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
- name: Install latest Rust toolchain
run: rustup update --no-self-update
- uses: Swatinem/rust-cache@v1
- name: Publish to Crates.io
uses: nick-fields/retry@v2
Expand All @@ -138,11 +126,8 @@ jobs:
if: startsWith(github.ref, 'refs/tags/release-lsp-async-stub-0')
steps:
- uses: actions/checkout@v4
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
- name: Install latest Rust toolchain
run: rustup update --no-self-update
- uses: Swatinem/rust-cache@v1
- name: Publish to Crates.io
uses: nick-fields/retry@v2
Expand All @@ -165,12 +150,10 @@ jobs:
with:
node-version: lts/*
registry-url: "https://registry.npmjs.org"
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
target: wasm32-unknown-unknown
- name: Install latest Rust toolchain
run: |
rustup update --no-self-update
rustup target add wasm32-unknown-unknown
- uses: Swatinem/rust-cache@v1
- name: Publish to NPM
uses: nick-fields/retry@v2
Expand All @@ -193,12 +176,10 @@ jobs:
with:
node-version: lts/*
registry-url: "https://registry.npmjs.org"
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
target: wasm32-unknown-unknown
- name: Install latest Rust toolchain
run: |
rustup update --no-self-update
rustup target add wasm32-unknown-unknown
- uses: Swatinem/rust-cache@v1
- name: Build Core
working-directory: js/core
Expand All @@ -224,12 +205,10 @@ jobs:
with:
node-version: lts/*
registry-url: "https://registry.npmjs.org"
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
target: wasm32-unknown-unknown
- name: Install latest Rust toolchain
run: |
rustup update --no-self-update
rustup target add wasm32-unknown-unknown
- uses: Swatinem/rust-cache@v1
- name: Build Core
working-directory: js/core
Expand All @@ -255,12 +234,10 @@ jobs:
with:
node-version: lts/*
registry-url: "https://registry.npmjs.org"
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
target: wasm32-unknown-unknown
- name: Install latest Rust toolchain
run: |
rustup update --no-self-update
rustup target add wasm32-unknown-unknown
- uses: Swatinem/rust-cache@v1
- name: Build Core
working-directory: js/core
Expand Down Expand Up @@ -492,11 +469,9 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
- name: Install latest Rust toolchain
run: |
rustup update --no-self-update
- uses: Swatinem/rust-cache@v1
- name: Install Node.js
uses: actions/setup-node@v4
Expand Down
10 changes: 4 additions & 6 deletions .github/workflows/site.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,10 @@ jobs:
- uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
override: true
- name: Install latest Rust toolchain
run: |
rustup update --no-self-update
rustup target add wasm32-unknown-unknown
- name: Setup Node
uses: actions/setup-node@v4
with:
Expand Down
Loading