Skip to content

Commit

Permalink
Merge pull request #239 from Fishrock123/ci-clippy-reduce
Browse files Browse the repository at this point in the history
ci: only run clippy as part of fmt
  • Loading branch information
yoshuawuyts authored Sep 19, 2020
2 parents a12d503 + c56db63 commit 743221d
Showing 1 changed file with 4 additions and 8 deletions.
12 changes: 4 additions & 8 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ jobs:
uses: actions-rs/toolchain@v1
with:
toolchain: ${{ matrix.rust }}
components: clippy
override: true

- name: check
Expand All @@ -41,20 +40,14 @@ jobs:
command: check
args: --all --benches --bins --examples --tests --features "hyperium_http,unstable"

- name: clippy
uses: actions-rs/cargo@v1
with:
command: clippy
args: --tests --examples -- -D warnings

- name: tests
uses: actions-rs/cargo@v1
with:
command: test
args: --all --features "hyperium_http,unstable"

check_fmt_and_docs:
name: Checking fmt and docs
name: Checking fmt, clippy, and docs
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@master
Expand All @@ -64,6 +57,9 @@ jobs:
components: rustfmt, clippy
override: true

- name: clippy
run: cargo clippy --tests --examples -- -D warnings

- name: fmt
run: cargo fmt --all -- --check

Expand Down

0 comments on commit 743221d

Please sign in to comment.