Skip to content

remove the double quoting of commit messages in bump command#10576

Closed
riknoll wants to merge 2 commits intomasterfrom
dev/riknoll/remove-double-quoting
Closed

remove the double quoting of commit messages in bump command#10576
riknoll wants to merge 2 commits intomasterfrom
dev/riknoll/remove-double-quoting

Conversation

@riknoll
Copy link
Member

@riknoll riknoll commented May 12, 2025

this is breaking the regex in the github action

the extra quotes are unnecessary because child_process.spawn already handles that for the args you pass i believe

@riknoll riknoll requested review from a team and eanders-ms May 12, 2025 17:25
@riknoll
Copy link
Member Author

riknoll commented May 12, 2025

actually, i wonder if this is a windows vs bash thing? @eanders-ms when you were bumping before were you doing it from command prompt?

@riknoll
Copy link
Member Author

riknoll commented May 12, 2025

confirmed, quotes are required on command prompt in windows. closing this; i'll open a new pr with a more complete fix

@riknoll riknoll closed this May 12, 2025
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.

1 participant