Skip to content

Commit 379bf07

Browse files
jeherveobenland
andauthored
Changelogger: move draft & base branch check to script (#1656)
Co-authored-by: Konstantin Obenland <[email protected]>
1 parent ff5b695 commit 379bf07

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

.github/workflows/changelog.yml

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,14 @@ on:
1010
jobs:
1111
# Enforces the addition of a changelog entry (a file in the .github/changelog directory) every pull request.
1212
changelog:
13-
if: github.event.pull_request.draft == false
1413
runs-on: ubuntu-latest
1514
steps:
16-
- name: "Check for Skip Changelog label"
15+
- name: "Check for Skip Changelog label, or if the PR is a draft"
1716
id: check-skip-label
1817
uses: actions/github-script@v7
1918
with:
2019
script: |
21-
const { payload : { pull_request : { number, labels } } } = context;
20+
const { payload : { pull_request : { number, labels, draft = false, base : { ref } } } } = context;
2221
2322
// Check for Skip Changelog label
2423
core.debug( 'Changelog check: Check for Skip Changelog label' );
@@ -29,6 +28,12 @@ jobs:
2928
if ( hasSkipLabel ) {
3029
core.info( `Skipping changelog requirement for this PR (#${ number }) because of the "${ skipLabel }" label.` );
3130
core.setOutput( 'skip-changelog', 'true' );
31+
} else if ( draft ) {
32+
core.info( `Skipping changelog requirement for this PR (#${ number }) because it is a draft.` );
33+
core.setOutput( 'skip-changelog', 'true' );
34+
} else if ( ref !== 'trunk' ) {
35+
core.info( `Skipping changelog requirement for this PR (#${ number }) because it is not against the "trunk" branch.` );
36+
core.setOutput( 'skip-changelog', 'true' );
3237
} else {
3338
core.info( `No "${ skipLabel }" label found for PR #${ number }. Will check for changelog file.` );
3439
core.setOutput( 'skip-changelog', 'false' );

0 commit comments

Comments
 (0)