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
feat(publish): Add a "from-package" argument #1708
feat(publish): Add a "from-package" argument #1708
Conversation
257823c
to
0e63025
Compare
I'm not sure why the CI job for window/node10 failed, but I think I finished everything that I had left WIP. |
Hello @evocateur Any chance you can take a look at this PR anytime soon? My team is also running into similar issues and this PR would be super helpful. Currently, we are forced to basically roll back the git changes and remove git tags and what not on intermittent fails. This would definitely be super helpful for us. Thank you, |
@chriscasola it was an installation issue by travis itself. Could you retrigger the build? |
@chriscasola rebase from master the .travis.yml was changed completly! |
Hi,
|
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.
My apologies for the delay, this is looking good so far! Would you mind switching to pacote for the packument fetching? (the add command uses the .manifest()
method [here])
This would only apply in independent versioning mode, yes? In the default "fixed" versioning, the tag will already be present when a partial publish occurs. |
@evocateur No in both because it doesn't create a git-tag in "fixed" mode in the current implementation. Please have a look at lerna-workflow to better understand the goals with #1648 The following fork https://github.com/StarpTech/lerna/tree/feature/publish-from-package will show you the idea. |
Publish un-published releases by reading versions from the package.json files and publishing any that are not already available in the registry. Fixes lerna#1648
c1cfcc5
to
55823e4
Compare
@evocateur if the workflow described in #1648 is what we're trying to follow here, where would the git tags be created?
I updated this PR with your other feedback, thanks! |
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.
Looks great, thanks!
I'm okay with not making the tags, it's overcomplicating the issue and teetering closer to patterns that I consider outside Lerna's scope. |
It's very useful. 👍 |
This thread has been automatically locked because there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Publish un-published releases by reading versions from
the package.json files and publishing any that are not
already available in the registry.
Fixes #1648
Description
This is an initial attempt at implementing
lerna publish from-package
. The "from-package" option will inspect each package.json and determine if the version has been published to the registry. If it hasn't it will publish it.Motivation and Context
This change will allow users to prepare a release, review it and commit it, and then later publish it in CI.
The change will also make it much easier to attempt to publish again if some or all of the packages failed to publish to the registry in a prior run of
lerna publish
.This may address #1648 and #1668
How Has This Been Tested?
Added unit tests.
Did the following on a test repo:
Types of changes
Checklist: