mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-06-09 09:34:57 +09:00
CI: Only run builds after successful linting
We can prevent running builds unnecessarily by requiring the linters to succeed first. If either the code or commit linter fails, it means the author of the PR needs to rework their branch and after pushing their changes, we need to do a full new CI run anyway.
This commit is contained in:
parent
086a921213
commit
6f69a445bd
Notes:
github-actions[bot]
2025-03-20 10:15:48 +00:00
Author: https://github.com/gmta
Commit: 6f69a445bd
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/4014
3 changed files with 13 additions and 2 deletions
9
.github/workflows/ci.yml
vendored
9
.github/workflows/ci.yml
vendored
|
@ -7,9 +7,18 @@ concurrency:
|
|||
cancel-in-progress: true
|
||||
|
||||
jobs:
|
||||
lint_code:
|
||||
name: 'Lint Code'
|
||||
uses: ./.github/workflows/lint-code.yml
|
||||
|
||||
lint_commits:
|
||||
name: 'Lint Commits'
|
||||
uses: ./.github/workflows/lint-commits.yml
|
||||
|
||||
# CI matrix - runs the job in lagom-template.yml with different configurations.
|
||||
Lagom:
|
||||
if: github.repository == 'LadybirdBrowser/ladybird'
|
||||
needs: [lint_code, lint_commits]
|
||||
|
||||
strategy:
|
||||
fail-fast: false
|
||||
|
|
3
.github/workflows/lint-code.yml
vendored
3
.github/workflows/lint-code.yml
vendored
|
@ -1,6 +1,7 @@
|
|||
name: Lint Code
|
||||
|
||||
on: [ push, pull_request ]
|
||||
# Used by ci.yml
|
||||
on: [workflow_call]
|
||||
|
||||
jobs:
|
||||
lint:
|
||||
|
|
3
.github/workflows/lint-commits.yml
vendored
3
.github/workflows/lint-commits.yml
vendored
|
@ -1,6 +1,7 @@
|
|||
name: Lint Commit Messages
|
||||
|
||||
on: [pull_request_target]
|
||||
# Used by ci.yml
|
||||
on: [workflow_call]
|
||||
|
||||
# Make sure to update Meta/lint-commit.sh to match this script when adding new checks!
|
||||
# (… but don't accept overlong 'fixup!' commit descriptions.)
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue