Skip to content

Commit

Permalink
Update PR template to avoid default regex self-match when copy-pasted…
Browse files Browse the repository at this point in the history
… to merge commit message (#468)

## Overview

Update PR template to avoid default regex self-match when copy-pasted to merge commit message.

Use roundabout comment description to avoid matching the regex.

## Changes

**Non-breaking changes**
- CI maintenance, PR template

**Breaking changes**
- n/a

## Related Issue / Discussion

- Fix PR not triggering CI.

## CI Operations


birdhouse_daccs_configs_branch: master
birdhouse_skip_ci: false
  • Loading branch information
fmigneault authored Sep 9, 2024
2 parents 327e6ea + f595ea7 commit a38b4c5
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,11 @@ Links to other issues or sources.
<!--
The test suite can be run using a different DACCS config with ``birdhouse_daccs_configs_branch: branch_name`` in the PR description.
To globally skip the test suite regardless of the commit message use ``birdhouse_skip_ci`` set to ``true`` in the PR description.
Note that using ``[skip ci]``, ``[ci skip]`` or ``[no ci]`` in the commit message will override ``birdhouse_skip_ci`` from the PR description.
Using ``[<cmd>]`` (with the brackets) where ``<cmd> = skip ci`` in the commit message will override ``birdhouse_skip_ci`` from the PR description.
Such commit command can be used to override the PR description behavior for a specific commit update.
However, a commit message cannot 'force run' a PR which the description turns off the CI.
To run the CI, the PR should instead be updated with a ``true`` value, and a running message can be posted in following PR comments to trigger tests once again.
-->

birdhouse_daccs_configs_branch: master
Expand Down

0 comments on commit a38b4c5

Please sign in to comment.