Skip to content

Commit 42289a7

Browse files
fix: gha security injection
1 parent 0f283d8 commit 42289a7

File tree

2 files changed

+5
-29
lines changed

2 files changed

+5
-29
lines changed

.github/workflows/changelog.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ jobs:
2727
name: "Check Changelog Entry"
2828
runs-on: ubuntu-latest
2929
concurrency:
30-
group: changelog-${{ github.head_ref }}
30+
group: changelog-${{ github.event.number }}
3131
cancel-in-progress: true
3232

3333
steps:
@@ -41,6 +41,8 @@ jobs:
4141
4242
- name: "Check for changelog entry"
4343
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
44+
env:
45+
CHANGELOG_CHANGES: ${{steps.changelog.outputs.changelog}}
4446
with:
4547
script: |
4648
async function createOrUpdateChangelogComment(commentDetails, deleteComment) {
@@ -90,7 +92,8 @@ jobs:
9092
}
9193
}
9294
93-
const changelogChangesPresent = ${{steps.changelog.outputs.changelog}};
95+
// Safely get the changelog output from environment variable
96+
const changelogChangesPresent = process.env.CHANGELOG_CHANGES === 'true';
9497
9598
const prAuthor = context.payload.pull_request.user.login;
9699
if (prAuthor === 'dependabot[bot]') {

.github/workflows/merged-pr.yml

Lines changed: 0 additions & 27 deletions
This file was deleted.

0 commit comments

Comments
 (0)