Skip to content

Commit

Permalink
add dependencies between workflows (#41)
Browse files Browse the repository at this point in the history
* QOL changes to the CI - dependency between formatting and nvidia-gpu tests, cancel ongoing tests on push
---------

Co-authored-by: Siddharth Singh <[email protected]>
  • Loading branch information
bhatele and siddharth9820 authored Oct 17, 2023
1 parent fd94ee3 commit e447012
Show file tree
Hide file tree
Showing 3 changed files with 35 additions and 14 deletions.
18 changes: 18 additions & 0 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
name: ci

on:
push:
branches: [ develop ]
pull_request:
branches: [ develop ]

concurrency:
group: ci-${{github.ref}}-${{github.event.pull_request.number || github.run_number}}
cancel-in-progress: true

jobs:
formatting:
uses: ./.github/workflows/formatting.yaml
nvidia-gpu:
needs: [ formatting ]
uses: ./.github/workflows/nvidia-rtx-3090-tests.yaml
Original file line number Diff line number Diff line change
@@ -1,18 +1,16 @@
name: formatting tests

on:
push:
branches: [ develop ]
pull_request:
branches: [ develop ]
workflow_dispatch:
workflow_call:

concurrency:
group: unit_tests-${{github.ref}}-${{github.event.pull_request.number || github.run_number}}
cancel-in-progress: true

jobs:
formatting:
runs-on: ${{ matrix.os }}

strategy:
matrix:
os: [ubuntu-latest, macos-latest]
runs-on: [ubuntu-latest]

steps:
- uses: actions/checkout@v2
Expand All @@ -34,3 +32,4 @@ jobs:
run: |
black --diff --check .
flake8
Original file line number Diff line number Diff line change
@@ -1,10 +1,12 @@
name: nvidia-rtx-3090 tests

on:
push:
branches: [ develop ]
pull_request:
branches: [ develop ]
workflow_dispatch:
workflow_call:

concurrency:
group: unit_tests-${{github.ref}}-${{github.event.pull_request.number || github.run_number}}
cancel-in-progress: true

jobs:
inter-layer:
Expand All @@ -13,7 +15,8 @@ jobs:
strategy:
matrix:
ginter: [ 1, 2 ]
memopt: [ '0', '1' ]
memopt: [ '0', '1' ]

steps:
- uses: actions/checkout@v3
- name: Install AxoNN
Expand All @@ -36,6 +39,7 @@ jobs:
intra-layer:
runs-on: [ nvidia ]

steps:
- uses: actions/checkout@v3
- name: Install AxoNN
Expand Down

0 comments on commit e447012

Please sign in to comment.