Skip to content

Commit

Permalink
fix: update documentation of baseline workflow
Browse files Browse the repository at this point in the history
  • Loading branch information
jnsbck committed Dec 2, 2024
1 parent 136ffc2 commit 1329d0a
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/update_regression_baseline.yml
Original file line number Diff line number Diff line change
Expand Up @@ -84,10 +84,11 @@ jobs:
fi
NEW_BASELINE=1 pytest -m regression
# The following will not work if the PR is initiated from a fork. In this case
# we create a new PR to update the baseline (see below).
# Only run if baseline generation succeeded
# Pushing to the PR branch does not work if the PR is initiated from a fork. This is
# because the GITHUB_TOKEN has read-only access by default for workflows triggered by
# fork PRs. Hence we have to create a new PR to update the baseline (see below).
- name: Commit and push to PR branch (non-fork)
# Only run if baseline generation succeeded
if: success() && steps.update-baseline.outcome == 'success' && !fromJson(steps.check-fork.outputs.result)
run: |
git add -f tests/regression_test_baselines.json # since it's in .gitignore
Expand Down

0 comments on commit 1329d0a

Please sign in to comment.