Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

New tag argument to control prefix of version #11

Merged
merged 2 commits into from Oct 12, 2021

Conversation

h4sh3d
Copy link
Contributor

@h4sh3d h4sh3d commented Oct 5, 2021

Fix #10

Add an new input tag and remove version as required option to allow
smooth transition to the new tag argument later. If tag is set version
argument is ignored, if version argument is set a warning is emitted to
transition to tag and tag is set to its value.

Internally tag is added along version to differentiate the version
(strict semver) and the tag (potentially 'v' prefixed semver). If the
tag is prefixed, version is set as the non-prefixed tag value.

Copy link
Owner

@thomaseizinger thomaseizinger left a comment

Choose a reason for hiding this comment

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

Very nice, thank you for tackling this!

One comment about spelling (lol), otherwise LGTM!

src/getInputs.ts Outdated Show resolved Hide resolved
@h4sh3d h4sh3d force-pushed the add-tag-argument branch 2 times, most recently from 048580b to 7e9f6cc Compare October 7, 2021 07:22
Add an new input `tag` and remove `version` as required option to allow
smooth transition to the new `tag` argument later. If tag is set version
argument is ignored, if version argument is set a warning is emitted to
transition to tag and tag is set to its value.

Internally tag is added along version to differentiate the version
(strict semver) and the tag (potentially 'v' prefixed semver). If the
tag is prefixed, version is set as the non-prefixed tag value.
@thomaseizinger
Copy link
Owner

Thanks!

@thomaseizinger thomaseizinger merged commit 5bc2328 into thomaseizinger:master Oct 12, 2021
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.

Add option to control prefix of version in compare urls
2 participants