ci: test both push and pull_request events
Some checks failed
/ check (treefmt) (pull_request) Successful in 24s
/ report-size-push (pull_request) Failing after 1s
/ report-size-pr (pull_request) Failing after 2s
/ report-download-check-push (pull_request) Failing after 1s
/ report-download-check-pr (pull_request) Failing after 1s
Some checks failed
/ check (treefmt) (pull_request) Successful in 24s
/ report-size-push (pull_request) Failing after 1s
/ report-size-pr (pull_request) Failing after 2s
/ report-download-check-push (pull_request) Failing after 1s
/ report-download-check-pr (pull_request) Failing after 1s
I think we have a bug with pull_request events so...
This commit is contained in:
parent
838f205020
commit
c68e4deaad
1 changed files with 41 additions and 6 deletions
|
@ -1,5 +1,7 @@
|
||||||
on:
|
on:
|
||||||
push:
|
push:
|
||||||
|
branches: [main]
|
||||||
|
pull_request:
|
||||||
jobs:
|
jobs:
|
||||||
check:
|
check:
|
||||||
runs-on: nixos
|
runs-on: nixos
|
||||||
|
@ -14,9 +16,10 @@ jobs:
|
||||||
nix --version
|
nix --version
|
||||||
# shellcheck disable=SC2016
|
# shellcheck disable=SC2016
|
||||||
nix build --print-build-logs '.#checks.x86_64-linux.${{ matrix.check }}'
|
nix build --print-build-logs '.#checks.x86_64-linux.${{ matrix.check }}'
|
||||||
report-size:
|
report-size-push:
|
||||||
runs-on: nixos
|
runs-on: nixos
|
||||||
needs: check
|
needs: check
|
||||||
|
if: github.event_name == 'push'
|
||||||
steps:
|
steps:
|
||||||
- uses: "https://git.salame.cl/actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683" # v4
|
- uses: "https://git.salame.cl/actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683" # v4
|
||||||
- run: nix --version
|
- run: nix --version
|
||||||
|
@ -24,18 +27,38 @@ jobs:
|
||||||
uses: ./
|
uses: ./
|
||||||
with:
|
with:
|
||||||
# Create a comment on the associated PR
|
# Create a comment on the associated PR
|
||||||
comment-on-pr: ${{ github.ref_name != 'main' }}
|
comment-on-pr: 'false'
|
||||||
|
# Generate artifacts on main (to speed up comparisons)
|
||||||
|
# generate-artifact: ${{ github.ref_name == 'main' }}
|
||||||
|
# Always generate artifacts for testing purposes
|
||||||
|
generate-artifact: 'true'
|
||||||
|
# This job's name (so we can find the previous artifacts)
|
||||||
|
job-name: report-size-push
|
||||||
|
report-size-pr:
|
||||||
|
runs-on: nixos
|
||||||
|
needs: check
|
||||||
|
if: github.event_name == 'pull_request'
|
||||||
|
steps:
|
||||||
|
- uses: "https://git.salame.cl/actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683" # v4
|
||||||
|
- run: nix --version
|
||||||
|
- name: Create Size Report
|
||||||
|
uses: ./
|
||||||
|
with:
|
||||||
|
# Create a comment on the associated PR
|
||||||
|
comment-on-pr: 'true'
|
||||||
# Generate artifacts on main (to speed up comparisons)
|
# Generate artifacts on main (to speed up comparisons)
|
||||||
# generate-artifact: ${{ github.ref_name == 'main' }}
|
# generate-artifact: ${{ github.ref_name == 'main' }}
|
||||||
# Always generate artifacts for testing purposes
|
# Always generate artifacts for testing purposes
|
||||||
generate-artifact: 'true'
|
generate-artifact: 'true'
|
||||||
# Generate comparisons to main
|
# Generate comparisons to main
|
||||||
do-comparison: 'true'
|
do-comparison: 'true'
|
||||||
# This job's name (so we can find the artifacts)
|
# Get the previous artifacts from report-size-push (since those run on main)
|
||||||
job-name: report-size
|
job-name: report-size-push
|
||||||
report-download-check:
|
artifact-name: report-pr.json
|
||||||
|
report-download-check-push:
|
||||||
runs-on: nixos
|
runs-on: nixos
|
||||||
needs: report-size
|
needs: report-size-push
|
||||||
|
if: github.event_name == 'push'
|
||||||
steps:
|
steps:
|
||||||
- name: Download previous report
|
- name: Download previous report
|
||||||
uses: https://git.salame.cl/actions/download-artifact@d8d0a99033603453ad2255e58720b460a0555e1e # v4
|
uses: https://git.salame.cl/actions/download-artifact@d8d0a99033603453ad2255e58720b460a0555e1e # v4
|
||||||
|
@ -44,3 +67,15 @@ jobs:
|
||||||
- name: Verify report exists
|
- name: Verify report exists
|
||||||
run: |
|
run: |
|
||||||
cat report.json
|
cat report.json
|
||||||
|
report-download-check-pr:
|
||||||
|
runs-on: nixos
|
||||||
|
needs: report-size-pr
|
||||||
|
if: github.event_name == 'pull_request'
|
||||||
|
steps:
|
||||||
|
- name: Download previous report
|
||||||
|
uses: https://git.salame.cl/actions/download-artifact@d8d0a99033603453ad2255e58720b460a0555e1e # v4
|
||||||
|
with:
|
||||||
|
name: report-pr.json
|
||||||
|
- name: Verify report exists
|
||||||
|
run: |
|
||||||
|
cat report-pr.json
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue