Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix git_clone where branch name has forward-slash #17359

Merged
merged 2 commits into from
Mar 4, 2025
Merged

fix git_clone where branch name has forward-slash #17359

merged 2 commits into from
Mar 4, 2025

Conversation

zzstoatzz
Copy link
Collaborator

closes #17355

@github-actions github-actions bot added the enhancement An improvement of an existing feature label Mar 4, 2025
@zzstoatzz zzstoatzz mentioned this pull request Mar 4, 2025
Closed
@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Mar 4, 2025

CodSpeed Performance Report

Merging #17359 will not alter performance

Comparing nate/branch-name (fe4069d) with main (8089cb5)

Summary

✅ 2 untouched benchmarks

desertaxle
desertaxle approved these changes Mar 4, 2025
View reviewed changes
Copy link
Member

@desertaxle desertaxle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@zzstoatzz zzstoatzz merged commit b657c9c into main Mar 4, 2025
46 checks passed
@zzstoatzz zzstoatzz deleted the nate/branch-name branch March 4, 2025 16:54
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@desertaxle desertaxle desertaxle approved these changes

@cicdw cicdw Awaiting requested review from cicdw cicdw is a code owner

Assignees
No one assigned
Labels
enhancement An improvement of an existing feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Correctly escape branch names with forward slash ('/')
2 participants
@zzstoatzz @desertaxle