Skip to content

Commit

Permalink
fix: ci.yml remove unused stuff
Browse files Browse the repository at this point in the history
- merge_group isn't relevant
- no python tests here
- not sure why we need all-tests

commit-id:6e45706e
  • Loading branch information
Gilad Chase committed Mar 20, 2024
1 parent 416f77c commit 0884ebd
Showing 1 changed file with 0 additions and 31 deletions.
31 changes: 0 additions & 31 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,6 @@ on:
- auto_merge_enabled
- edited

merge_group:
types: [checks_requested]

jobs:
commitlint:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -59,18 +56,6 @@ jobs:
- uses: Swatinem/rust-cache@v2
- run: scripts/clippy.sh

run-python-tests:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v2
with:
python-version: '3.9'
- run: |
python -m pip install --upgrade pip
pip install pytest
- run: pytest scripts/merge_paths_test.py

run-tests:
runs-on: ubuntu-latest
steps:
Expand All @@ -96,19 +81,3 @@ jobs:
cargo-udeps-*/cargo-udeps udeps
env:
RUSTUP_TOOLCHAIN: nightly-2024-01-12


all-tests:
runs-on: ubuntu-latest
needs:
- clippy
- commitlint
- format
- run-python-tests
- run-tests
- udeps
steps:
- name: Decide whether all the needed jobs succeeded or failed
uses: re-actors/alls-green@v1.2.2
with:
jobs: ${{ toJSON(needs) }}

0 comments on commit 0884ebd

Please sign in to comment.