-
Notifications
You must be signed in to change notification settings - Fork 285
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
Adds it-tools #39684
base: main
Are you sure you want to change the base?
Adds it-tools #39684
Conversation
Signed-off-by: Srishti Hegde <[email protected]>
Signed-off-by: Srishti Hegde <[email protected]>
Signed-off-by: Srishti Hegde <[email protected]>
Signed-off-by: Srishti Hegde <[email protected]>
Signed-off-by: Srishti Hegde <[email protected]>
Signed-off-by: Srishti Hegde <[email protected]>
Signed-off-by: Srishti Hegde <[email protected]>
Signed-off-by: Srishti Hegde <[email protected]>
# The version includes a commit from one of the pull requests that contribute to the new version | ||
git clone --depth=1 https://github.com/CorentinTh/it-tools.git | ||
cd it-tools | ||
package_version="${{package.version}}" | ||
git fetch --tags | ||
latest_tag=$(git tag -l "v${package_version}-*" | sort -V | tail -n 1) | ||
|
||
git checkout ${latest_tag} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we create a new ReleaseMonitor and use git-checkout` pipeline here instead? That way we should be able to access latest tag.
|
||
- runs: | | ||
cd it-tools | ||
npm install -g pnpm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we still need this since we already have pnpm
in the build-time deps.
Fixes:
Related:
Pre-review Checklist
For new package PRs only
endoflife.date
)For version bump PRs
epoch
field is reset to 0