Skip to content

Commit

Permalink
Remove duplicate CI checks that already have dedicated jobs (FuelLabs…
Browse files Browse the repository at this point in the history
…#2161)

I noticed a bunch of our CI jobs were were duplicating existing checks.
In particular, there was a lot of copy-paste steps of installing forc
and the forc-fmt plugin, even when unused in the following checks.

This doesn't improve the real bottleneck (our stdlib test job), but it's
a start.
  • Loading branch information
mitchmindtree authored Jun 29, 2022
1 parent 67e67b1 commit 987f950
Showing 1 changed file with 0 additions and 80 deletions.
80 changes: 0 additions & 80 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -162,11 +162,6 @@ jobs:
with:
command: install
args: --debug --path ./forc
- name: Install Forc fmt
uses: actions-rs/cargo@v1
with:
command: install
args: --debug --path ./forc-plugins/forc-fmt
- name: Initialize test project
run: forc init test-proj
- name: Update project forc manifest to use local sway-lib-std
Expand All @@ -191,21 +186,6 @@ jobs:
profile: minimal
toolchain: stable
- uses: Swatinem/rust-cache@v1
- name: Install Forc
uses: actions-rs/cargo@v1
with:
command: install
args: --debug --path ./forc
- name: Install Forc fmt
uses: actions-rs/cargo@v1
with:
command: install
args: --debug --path ./forc-plugins/forc-fmt
- name: Build Sway examples
uses: actions-rs/cargo@v1
with:
command: run
args: --bin examples-checker build --all-examples
- uses: actions-rs/cargo@v1
name: Cargo Build Workspace
with:
Expand All @@ -223,21 +203,6 @@ jobs:
with:
toolchain: stable
- uses: Swatinem/rust-cache@v1
- name: Install Forc
uses: actions-rs/cargo@v1
with:
command: install
args: --debug --path ./forc
- name: Install Forc fmt
uses: actions-rs/cargo@v1
with:
command: install
args: --debug --path ./forc-plugins/forc-fmt
- name: Build Sway examples
uses: actions-rs/cargo@v1
with:
command: run
args: --bin examples-checker build --all-examples
- name: Check Clippy Linter
uses: actions-rs/cargo@v1
with:
Expand Down Expand Up @@ -311,21 +276,6 @@ jobs:
profile: minimal
toolchain: stable
- uses: Swatinem/rust-cache@v1
- name: Install Forc
uses: actions-rs/cargo@v1
with:
command: install
args: --debug --path ./forc
- name: Install Forc fmt
uses: actions-rs/cargo@v1
with:
command: install
args: --debug --path ./forc-plugins/forc-fmt
- name: Build Sway examples
uses: actions-rs/cargo@v1
with:
command: run
args: --locked --bin examples-checker build --all-examples
- name: Build All Tests
run: cd test/src/sdk-harness && bash build.sh --locked && cd ../../../
- name: Cargo Test sway-lib-std
Expand All @@ -344,21 +294,6 @@ jobs:
profile: minimal
toolchain: stable
- uses: Swatinem/rust-cache@v1
- name: Install Forc
uses: actions-rs/cargo@v1
with:
command: install
args: --debug --path ./forc
- name: Install Forc fmt
uses: actions-rs/cargo@v1
with:
command: install
args: --debug --path ./forc-plugins/forc-fmt
- name: Build Sway examples
uses: actions-rs/cargo@v1
with:
command: run
args: --bin examples-checker build --all-examples
- name: Run tests
uses: actions-rs/cargo@v1
with:
Expand All @@ -376,21 +311,6 @@ jobs:
toolchain: nightly
default: true
- uses: Swatinem/rust-cache@v1
- name: Install Forc
uses: actions-rs/cargo@v1
with:
command: install
args: --debug --path ./forc
- name: Install Forc fmt
uses: actions-rs/cargo@v1
with:
command: install
args: --debug --path ./forc-plugins/forc-fmt
- name: Build Sway examples
uses: actions-rs/cargo@v1
with:
command: run
args: --locked --bin examples-checker build --all-examples
- name: Install cargo-udeps
uses: actions-rs/cargo@v1
with:
Expand Down

0 comments on commit 987f950

Please sign in to comment.