Skip to content

fix(scan): accept a VERSION pin without the v prefix - #35

Open
Paramjeet-singh-neu wants to merge 1 commit into
trustabl:mainfrom
Paramjeet-singh-neu:fix/version-tag-prefix
Open

fix(scan): accept a VERSION pin without the v prefix#35
Paramjeet-singh-neu wants to merge 1 commit into
trustabl:mainfrom
Paramjeet-singh-neu:fix/version-tag-prefix

Conversation

@Paramjeet-singh-neu

Copy link
Copy Markdown

Summary

Release tags are v0.1.7. Pinning VERSION=0.1.7 (no v) built:

https://github.com/trustabl/trustabl/releases/download/0.1.7/trustabl_0.1.7_…

which 404s against tag v0.1.7. The wrapper now prepends v when the pin has no prefix. latest is unchanged (the API already returns v0.1.7).

The README example was v0.5.0, a tag that does not exist. Replaced with v0.1.7.

Test plan

  • bash test/version-tag.sh0.1.7v0.1.7; v0.1.7 stays
  • Pin VERSION=0.1.7 on a CodeBuild project and confirm the download URL includes /download/v0.1.7/

Made with Cursor

Release tags are v0.1.7. VERSION=0.1.7 built a download URL under
/download/0.1.7/ which 404s. Prepend v when the pin has no prefix.
The README example was v0.5.0, a tag that does not exist; use v0.1.7.

Co-authored-by: Cursor <cursoragent@cursor.com>
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