please icon indicating copy to clipboard operation
please copied to clipboard

`github_repo`: strip `v` prefix if `revision` is any version number

Open chrisnovakovic opened this issue 1 year ago • 3 comments

github_repo currently strips the v prefix from the file names in the zip file downloaded from GitHub if revision is a semver, since GitHub silently does the same if asked to archive the code for semver-like tags. It looks like this logic extends to more than just semvers, though - the same behaviour has been observed for strings that are simple dotted version numbers with any number of components (e.g. v1.1.10.1). Account for this in the prefix-stripping logic.

Fixes #2829.

chrisnovakovic avatar Jun 19 '23 14:06 chrisnovakovic

This issue has been automatically marked as stale because it has not had any recent activity in the past 90 days. It will be closed if no further activity occurs. If you require additional support, please reply to this message. Thank you for your contributions.

stale[bot] avatar Oct 15 '23 11:10 stale[bot]

This issue has been automatically marked as stale because it has not had any recent activity in the past 90 days. It will be closed if no further activity occurs. If you require additional support, please reply to this message. Thank you for your contributions.

Not stale, but I'm going to think of a more efficient way to implement this.

chrisnovakovic avatar Nov 04 '23 23:11 chrisnovakovic

This issue has been automatically marked as stale because it has not had any recent activity in the past 90 days. It will be closed if no further activity occurs. If you require additional support, please reply to this message. Thank you for your contributions.

stale[bot] avatar Mar 17 '24 15:03 stale[bot]