Skip to content

Commit

Permalink
build: catch up all the new best practices (#15)
Browse files Browse the repository at this point in the history
Signed-off-by: tison <[email protected]>
  • Loading branch information
tisonkun authored Oct 28, 2024
1 parent 59f08bd commit b617154
Show file tree
Hide file tree
Showing 24 changed files with 598 additions and 628 deletions.
13 changes: 13 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,17 @@ jobs:
tool: typos-cli,taplo-cli,hawkeye
- run: cargo x lint

build:
name: Check release build
strategy:
matrix:
os: [ ubuntu-22.04, macos-14, windows-2022 ]
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v4
- uses: Swatinem/rust-cache@v2
- run: cargo build --workspace --release --tests --examples --benches --bins

test:
name: Run tests with testcontainers
runs-on: ubuntu-22.04
Expand All @@ -58,13 +69,15 @@ jobs:
runs-on: ubuntu-22.04
if: ${{ always() }}
needs:
- build
- check
- test
steps:
- name: Guardian
run: |
if [[ ! ( \
"${{ needs.check.result }}" == "success" \
&& "${{ needs.build.result }}" == "success" \
&& "${{ needs.test.result }}" == "success" \
) ]]; then
echo "Required jobs haven't been completed successfully."
Expand Down
4 changes: 0 additions & 4 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -117,10 +117,6 @@ jobs:
- uses: actions/checkout@v4
with:
submodules: recursive
- uses: swatinem/rust-cache@v2
with:
key: ${{ join(matrix.targets, '-') }}
cache-provider: ${{ matrix.cache_provider }}
- name: Install cargo-dist
run: ${{ matrix.install_dist }}
# Get the dist-manifest
Expand Down
Loading

0 comments on commit b617154

Please sign in to comment.