Skip to content

fix: decode branch names when parsing GitHub URLs#3

Merged
dwilding merged 3 commits intomainfrom
decode-branch-urls
Aug 29, 2025
Merged

fix: decode branch names when parsing GitHub URLs#3
dwilding merged 3 commits intomainfrom
decode-branch-urls

Conversation

@dwilding
Copy link
Copy Markdown
Owner

@dwilding dwilding commented Aug 29, 2025

gimmegit crashed on this GitHub URL:

gimmegit https://github.com/james-garner-canonical/charmlibs/tree/25-09%2Bdocs%2Bbuild-reference-docs-separately

The crash happened because gimmegit didn't decode %2B to +, so was trying to check out a branch literally called 25-09%2Bdocs%2Bbuild-reference-docs-separately. This PR fixes the bug.

Drive-bys:

  • Change version number to 0.0.15.dev0
  • Remove unused snapshot_name fixture from some tests

@dwilding dwilding merged commit 297364a into main Aug 29, 2025
2 checks passed
@dwilding dwilding deleted the decode-branch-urls branch August 29, 2025 12:33
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