Skip to content

Only quote commit messages if bumping in windows command prompt#10581

Merged
riknoll merged 3 commits intomasterfrom
dev/riknoll/remove-double-quoting
May 12, 2025
Merged

Only quote commit messages if bumping in windows command prompt#10581
riknoll merged 3 commits intomasterfrom
dev/riknoll/remove-double-quoting

Conversation

@riknoll
Copy link
Member

@riknoll riknoll commented May 12, 2025

child_process.spawn handles the escaping of arguments in bash, but not in windows. this meant that we were getting double quotes in prs made from WSL (see #10575) which in turn broke the regex in our github action that detected whether the merged commit should be tagged or not.

@riknoll riknoll requested a review from a team May 12, 2025 18:01
@riknoll riknoll merged commit 728ccaa into master May 12, 2025
20 checks passed
@riknoll riknoll deleted the dev/riknoll/remove-double-quoting branch May 12, 2025 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants