-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
fix(js): allow publishable library to run release command #29775
fix(js): allow publishable library to run release command #29775
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
View your CI Pipeline Execution ↗ for commit c085917.
☁️ Nx Cloud last updated this comment at |
dcfe278
to
ce3fe9e
Compare
a71555f
to
0762eb6
Compare
fe594f1
to
0762eb6
Compare
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.
Mainly +1'ing Leo's feedback but also a note about needing to support the workspaces + TS project references setup
3d0d1bd
to
25b881e
Compare
25b881e
to
1a874c9
Compare
37f3a20
to
0759294
Compare
0759294
to
506781d
Compare
506781d
to
c085917
Compare
<!-- Please make sure you have read the submission guidelines before posting an PR --> <!-- https://github.com/nrwl/nx/blob/master/CONTRIBUTING.md#-submitting-a-pr --> <!-- Please make sure that your commit message follows our format --> <!-- Example: `fix(nx): must begin with lowercase` --> <!-- If this is a particularly complex change or feature addition, you can request a dedicated Nx release for this pull request branch. Mention someone from the Nx team or the `@nrwl/nx-pipelines-reviewers` and they will confirm if the PR warrants its own release for testing purposes, and generate it for you if appropriate. --> ## Current Behavior <!-- This is the behavior we have today --> - there is no option `fallbackCurrentVersionResolver: 'disk',` - can't run release for newly created publishable libraries ## Expected Behavior <!-- This is the behavior we should expect with the changes in this PR --> - move code to add release for publishable libraries into its own file to be reused by other stacks - add `fallbackCurrentVersionResolver: 'disk',` to project's release.version. generatorOptions ## Related Issue(s) <!-- Please link the issue being fixed so it gets closed when this is merged. --> Fixes #29723
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
fallbackCurrentVersionResolver: 'disk',
Expected Behavior
fallbackCurrentVersionResolver: 'disk',
to project's release.version. generatorOptionsRelated Issue(s)
Fixes #29723