Skip to content

Commit 623e264

Browse files
aduh95targos
authored andcommitted
tools: ignore CVE mention when linting release proposals
PR-URL: #59037 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
1 parent 467dbd3 commit 623e264

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

.github/workflows/lint-release-proposal.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ jobs:
3939
EXPECTED_TRAILER="^$GITHUB_SERVER_URL/$GITHUB_REPOSITORY/pull/[[:digit:]]+\$"
4040
echo "Expected trailer format: $EXPECTED_TRAILER"
4141
PR_URL="$(git --no-pager log -1 --format='%(trailers:key=PR-URL,valueonly)')"
42-
echo "Actual: $ACTUAL"
42+
echo "Actual: $PR_URL"
4343
echo "$PR_URL" | grep -E -q "$EXPECTED_TRAILER"
4444
4545
PR_HEAD="$(gh pr view "$PR_URL" --json headRefOid -q .headRefOid)"

tools/actions/lint-release-proposal-commit-list.mjs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ if (commitListingStart === -1) {
3434
// Normalize for consistent comparison
3535
commitList = commitList
3636
.replaceAll('**(SEMVER-MINOR)** ', '')
37+
.replaceAll(/(?<= - )\*\*\(CVE-\d{4}-\d+\)\*\* (?=\*\*)/g, '')
3738
.replaceAll('\\', '');
3839

3940
let expectedNumberOfCommitsLeft = commitList.match(/\n\* \[/g)?.length ?? 0;

0 commit comments

Comments
 (0)