Skip to content

Commit

Permalink
yamllint cleanup lint.yml test.yml
Browse files Browse the repository at this point in the history
NFC fix most `yamllint` errors and warnings in
`lint.yml` and `test.yml`.
  • Loading branch information
jtmoon79 committed May 29, 2023
1 parent c01e3a7 commit 1fa730c
Show file tree
Hide file tree
Showing 2 changed files with 37 additions and 10 deletions.
15 changes: 11 additions & 4 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
# lint.yml
---
name: lint

on:
Expand All @@ -18,16 +20,20 @@ jobs:
- run: cargo fmt --check -- --color=always
- run: cargo fmt --check --manifest-path fuzz/Cargo.toml
- run: cargo clippy --color=always -- -D warnings
- run: cargo clippy --color=always --target x86_64-pc-windows-msvc -- -D warnings
- run: cargo clippy --manifest-path fuzz/Cargo.toml --color=always -- -D warnings
- run: |
cargo clippy --color=always --target x86_64-pc-windows-msvc \
-- -D warnings
- run: |
cargo clippy --manifest-path fuzz/Cargo.toml --color=always \
-- -D warnings
env:
RUSTFLAGS: "-Dwarnings"
cargo-deny:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: EmbarkStudios/cargo-deny-action@v1
- uses: actions/checkout@v3
- uses: EmbarkStudios/cargo-deny-action@v1

check-doc:
runs-on: ubuntu-latest
Expand All @@ -39,3 +45,4 @@ jobs:
- run: cargo doc --all-features --no-deps
env:
RUSTDOCFLAGS: -Dwarnings
...
32 changes: 26 additions & 6 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
# test.yml
---
name: All Tests and Builds

on:
Expand Down Expand Up @@ -45,9 +47,18 @@ jobs:
with:
toolchain: 1.56.1
- uses: Swatinem/rust-cache@v2
# run --lib and --doc to avoid the long running integration tests which are run elsewhere
- run: cargo test --lib --features unstable-locales,wasmbind,oldtime,clock,rustc-serialize,winapi,serde --color=always -- --color=always
- run: cargo test --doc --features unstable-locales,wasmbind,oldtime,clock,rustc-serialize,winapi,serde --color=always -- --color=always
# run --lib and --doc to avoid the long running integration tests
# which are run elsewhere
- run: |
cargo test --lib \
--features \
unstable-locales,wasmbind,oldtime,clock,rustc-serialize,winapi,serde \
--color=always -- --color=always
- run: |
cargo test --doc \
--features \
unstable-locales,wasmbind,oldtime,clock,rustc-serialize,winapi,serde \
--color=always -- --color=always
rust_versions:
strategy:
Expand All @@ -63,7 +74,8 @@ jobs:
- uses: Swatinem/rust-cache@v2
- run: cargo check --benches
- run: cargo check --manifest-path fuzz/Cargo.toml --all-targets
# run --lib and --doc to avoid the long running integration tests which are run elsewhere
# run --lib and --doc to avoid the long running integration tests
# which are run elsewhere
- run: cargo test --lib --all-features --color=always -- --color=always
- run: cargo test --doc --all-features --color=always -- --color=always

Expand All @@ -77,7 +89,11 @@ jobs:
- uses: dtolnay/rust-toolchain@stable
- uses: taiki-e/install-action@cargo-hack
- uses: Swatinem/rust-cache@v2
- run: cargo hack check --feature-powerset --optional-deps serde,rkyv --skip default --skip __internal_bench --skip __doctest --skip iana-time-zone --skip pure-rust-locales
- run: |
cargo hack check --feature-powerset --optional-deps serde,rkyv \
--skip default --skip __internal_bench --skip __doctest \
--skip iana-time-zone --skip pure-rust-locales
shell: bash
- run: cargo test --lib --no-default-features
- run: cargo test --doc --no-default-features

Expand Down Expand Up @@ -136,7 +152,10 @@ jobs:
targets: wasm32-unknown-unknown
- uses: taiki-e/install-action@cargo-hack
- uses: Swatinem/rust-cache@v2
- run: cargo hack check --feature-powerset --optional-deps serde,rkyv --skip default --skip __internal_bench --skip __doctest --skip iana-time-zone --skip pure-rust-locales
- run: |
cargo hack check --feature-powerset --optional-deps serde,rkyv \
--skip default --skip __internal_bench --skip __doctest \
--skip iana-time-zone --skip pure-rust-locales
cross-targets:
strategy:
Expand All @@ -158,3 +177,4 @@ jobs:
- run: cargo +nightly doc --all-features --no-deps
env:
RUSTDOCFLAGS: "-D warnings --cfg docsrs"
...

0 comments on commit 1fa730c

Please sign in to comment.