mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-09-13 21:12:26 +00:00
CI: Only run commit linter for pull requests, not pushes
This requires us to always run the CI job and check the individual jobs' results, since only having `needs:` will not work when `lint_commits` is potentially skipped.
This commit is contained in:
parent
6f69a445bd
commit
bf333eaea2
Notes:
github-actions[bot]
2025-03-20 10:40:26 +00:00
Author: https://github.com/gmta
Commit: bf333eaea2
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/4015
1 changed files with 6 additions and 1 deletions
7
.github/workflows/ci.yml
vendored
7
.github/workflows/ci.yml
vendored
|
@ -13,12 +13,17 @@ jobs:
|
||||||
|
|
||||||
lint_commits:
|
lint_commits:
|
||||||
name: 'Lint Commits'
|
name: 'Lint Commits'
|
||||||
|
if: ${{ github.event_name == 'pull_request' }}
|
||||||
uses: ./.github/workflows/lint-commits.yml
|
uses: ./.github/workflows/lint-commits.yml
|
||||||
|
|
||||||
# CI matrix - runs the job in lagom-template.yml with different configurations.
|
# CI matrix - runs the job in lagom-template.yml with different configurations.
|
||||||
Lagom:
|
Lagom:
|
||||||
if: github.repository == 'LadybirdBrowser/ladybird'
|
|
||||||
needs: [lint_code, lint_commits]
|
needs: [lint_code, lint_commits]
|
||||||
|
if: |
|
||||||
|
always()
|
||||||
|
&& github.repository == 'LadybirdBrowser/ladybird'
|
||||||
|
&& needs.lint_code.result == 'success'
|
||||||
|
&& (needs.lint_commits.result == 'skipped' || needs.lint_commits.result == 'success')
|
||||||
|
|
||||||
strategy:
|
strategy:
|
||||||
fail-fast: false
|
fail-fast: false
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue