Skip to content

Commit

Permalink
chore: ci workflow enhancements (#3778)
Browse files Browse the repository at this point in the history
* chore: include mac-os as env for test workflow

* chore: include mac-os as env for test-lint workflow

* chore: include mac-os as env for test-integrate workflow

* chore: update labeler mapping

* chore: include mac-os as env for binaries workflow

* keep release binary workflow only on ubuntu

---------

Co-authored-by: Danilo Pantani <[email protected]>
  • Loading branch information
salmad3 and Pantani authored Dec 1, 2023
1 parent b62a3e7 commit c35cf70
Show file tree
Hide file tree
Showing 4 changed files with 22 additions and 14 deletions.
18 changes: 9 additions & 9 deletions .github/labeler.yml
Original file line number Diff line number Diff line change
@@ -1,33 +1,33 @@
":robot: CI":
":component:ci":
- "!.github/labeler.yml"
- "!.github/labels.json"
- .github/**/*

":book: docs":
":component:docs":
- assets/**/*
- docs/**/*

":hammer: tools":
":component:tools":
- .actions/**/*
- .scripts/**/*

":package: packages":
":component:packages":
- ignite/pkg/**/*

":arrow_forward: cmd":
":component:cmd":
- ignite/cmd/**/*

":taxi: services":
":type:services":
- ignite/services/**/*

":bookmark: templates":
":component:templates":
- ignite/templates/**/*

":door: internal":
":type:internal":
- ignite/config/**/*
- ignite/internal/**/*
- ignite/version/**/*

":wrench: configs":
":component:configs":
- "!*.md"
- "*"
6 changes: 4 additions & 2 deletions .github/workflows/test-integration.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
name: Integration

on:
pull_request:
push:
Expand All @@ -24,14 +25,15 @@ jobs:
run: echo "matrix=$({ cd integration && find . -type d ! -name testdata -maxdepth 1 -print; } | tail -n +2 | cut -c 3- | jq -R . | jq -cs .)" >> $GITHUB_OUTPUT

integration:
name: test ${{ matrix.test-path }}
runs-on: ubuntu-latest
name: test ${{ matrix.test-path }} on ${{ matrix.os }}
runs-on: ${{ matrix.os }}
needs: pre-test
if: fromJSON(needs.pre-test.outputs.matrix)[0] != null
continue-on-error: false
strategy:
fail-fast: true
matrix:
os: [ubuntu-latest, macos-latest]
test-path: ${{fromJson(needs.pre-test.outputs.matrix)}}
steps:
- uses: actions/[email protected]
Expand Down
6 changes: 5 additions & 1 deletion .github/workflows/test-lint.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
name: Lint

on:
pull_request:
push:
Expand All @@ -15,7 +16,10 @@ concurrency:
jobs:
lint:
name: Lint Go code
runs-on: ubuntu-latest
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [ubuntu-latest, macos-latest]
timeout-minutes: 6
steps:
- uses: actions/[email protected]
Expand Down
6 changes: 4 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,10 @@ concurrency:

jobs:
test:
runs-on: ubuntu-latest
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [ubuntu-latest, macos-latest]
steps:
- uses: actions/[email protected]
- uses: technote-space/[email protected]
Expand Down Expand Up @@ -44,4 +47,3 @@ jobs:
file: ./coverage.txt
fail_ci_if_error: true
verbose: true

0 comments on commit c35cf70

Please sign in to comment.