CI: Refactor branch matching in jobs
Matching `master` can be a bit simpler and prevent invoking the job altogether. Additionally, we don't need an `always()` here: we don't have job dependencies and we want these jobs to be cancelable.
This commit is contained in:
parent
8ddd9ea5f4
commit
b00bb05b6b
2 changed files with 8 additions and 4 deletions
6
.github/workflows/ladybird-js-artifacts.yml
vendored
6
.github/workflows/ladybird-js-artifacts.yml
vendored
|
|
@ -1,6 +1,8 @@
|
|||
name: Package the js repl as a binary artifact
|
||||
|
||||
on: [push]
|
||||
on:
|
||||
push:
|
||||
branches: [master]
|
||||
|
||||
env:
|
||||
LADYBIRD_SOURCE_DIR: ${{ github.workspace }}
|
||||
|
|
@ -11,7 +13,7 @@ env:
|
|||
jobs:
|
||||
build-and-package:
|
||||
runs-on: ${{ matrix.os }}
|
||||
if: always() && github.repository == 'LadybirdBrowser/ladybird' && github.ref == 'refs/heads/master'
|
||||
if: github.repository == 'LadybirdBrowser/ladybird'
|
||||
strategy:
|
||||
fail-fast: false
|
||||
matrix:
|
||||
|
|
|
|||
6
.github/workflows/libjs-test262.yml
vendored
6
.github/workflows/libjs-test262.yml
vendored
|
|
@ -1,6 +1,8 @@
|
|||
name: Run test262 and test-wasm
|
||||
|
||||
on: [push]
|
||||
on:
|
||||
push:
|
||||
branches: [master]
|
||||
|
||||
env:
|
||||
LADYBIRD_SOURCE_DIR: ${{ github.workspace }}
|
||||
|
|
@ -9,7 +11,7 @@ env:
|
|||
jobs:
|
||||
run_and_update_results:
|
||||
runs-on: test262-runner
|
||||
if: always() && github.repository == 'LadybirdBrowser/ladybird' && github.ref == 'refs/heads/master'
|
||||
if: github.repository == 'LadybirdBrowser/ladybird'
|
||||
|
||||
concurrency: libjs-test262
|
||||
|
||||
|
|
|
|||
Loading…
Reference in a new issue