-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
More reliable canary bump #2171
Comments
|
I guess it will work, but it can ruin performance on a large size repo... |
Well, there's all sorts of fancy new flags in git fetch that could be used to mitigate that (assuming your CI host has a reasonably up-to-date |
Why not just get latest tags/versions from NPM? It's the source of truth after all, isn't it? Or like I said in the original post — let the CI job to define the increment. |
Hi Folks 👋 You may or may not know that lerna is now under the stewardship of Nrwl (announcement here #3121), a company with a long history of not just producing valuable open-source software (OSS), but also backing others (at the time of writing, Nrwl has donated over $50,000 to OSS it hasn't created, see https://opencollective.com/nx for full details). Quite simply, Nrwl ❤️ OSS, and is committed to making lerna the best it can be. We use it ourselves. In order to take this awesome project forward from its current state, it is important that we focus our finite resources on what is most important to lerna users in 2022. With that in mind, we have identified this issue as being potentially stale due to its age and/or lack of recent activity. Next steps: We want to give you some time to read through this comment and take action per one of the steps outlined below, so for the next 14 days we will not make any further updates to this issue. @kirill-konshin as the original author of this issue, we are looking to you to update us on the latest state of this as it relates to the latest version of lerna. Please choose one of the steps below, depending on what type of issue this is:
If we do not hear from @kirill-konshin on this thread within the next 14 days, we will automatically close this issue. If you are another user impacted by this issue but it ends up being closed as part of this process, we still want to hear from you! Please simply head over to our new issue templates and fill out all the requested details on the template which applies to your situation: https://github.com/lerna/lerna/issues/new/choose Thank you all for being a part of this awesome community, we could not be more excited to help move things forward from here 🙏 🚀 |
Hi Folks 👋 You will have seen in our message above that we at Nrwl are working really hard to bring the lerna repo up to date with what matters most to its community in June 2022. As previously stated in that message, because we have not heard from the original author of this issue within the last 14 days, we are now automatically closing it. If any users, including the original author, are still impacted by this issue then we still want to hear from you! All we ask is that you first update to the latest lerna ( https://github.com/lerna/lerna/issues/new/choose Many thanks again! 🙏 P.S. Over and above getting to grips with the repo, we have also been hard at work launching a new website, resolving all vulnerabilities, merging exciting new features and reigniting community PR contributions! 🚀 You can read our recent blog post to learn more about what we've been up to: |
As stated in #1583 (comment) Lerna cannot properly bump canary releases in shallow Git repo.
Currently we have to use something incremental and reliable, Travis's job number or Gitlab's pipeline IID works just fine:
Unfortunately this results in versions like
19.4.1-feature-npm.4247.1
where.1
is that improperly working increment, which is always the same.In order to fix this we can count previous releases in NPM under same
dist-tag
instead. Either inside Lerna or externally if you don't want to have this in your code. But then please provide a way to manually set thecanary-increment
duringpublish
:So as a summary, one of the options would work:
The text was updated successfully, but these errors were encountered: