Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(js): generate nx release config correctly for js libraries in new ts setup and set tags #28218

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

leosvelperez
Copy link
Member

@leosvelperez leosvelperez commented Oct 1, 2024

Current Behavior

When setting the Nx configuration for a JS library in the package.json file, the provided tags is not set and the relevant nx release configuration in nx.json is not set either.

Expected Behavior

When setting the Nx configuration for a JS library in the package.json file, the provided tags should be set and the relevant nx release configuration in nx.json should be configured.

Related Issue(s)

Fixes #

@leosvelperez leosvelperez requested a review from jaysoo October 1, 2024 10:01
@leosvelperez leosvelperez self-assigned this Oct 1, 2024
@leosvelperez leosvelperez requested a review from a team as a code owner October 1, 2024 10:01
Copy link

vercel bot commented Oct 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview Oct 2, 2024 7:49am

Copy link

github-actions bot commented Oct 1, 2024

🐳 We have a release for that!

This PR has a release associated with it. You can try it out using this command:

npx create-nx-workspace@0.0.0-pr-28218-5048fd1 my-workspace

Or just copy this version and use it in your own command:

0.0.0-pr-28218-5048fd1
Release details 📑
Published version 0.0.0-pr-28218-5048fd1
Triggered by @leosvelperez
Branch js/fix-library-config-in-package-json
Commit 5048fd1
Workflow run 11123470535

To request a new release for this pull request, mention someone from the Nx team or the @nrwl/nx-pipelines-reviewers.

@leosvelperez leosvelperez force-pushed the js/fix-library-config-in-package-json branch from 5048fd1 to d4ec1f7 Compare October 1, 2024 12:28
@leosvelperez leosvelperez marked this pull request as draft October 1, 2024 13:01
@leosvelperez leosvelperez force-pushed the js/fix-library-config-in-package-json branch from d4ec1f7 to cc10cac Compare October 1, 2024 13:14
@leosvelperez leosvelperez force-pushed the js/fix-library-config-in-package-json branch from cc10cac to 2a3550e Compare October 2, 2024 07:46
@leosvelperez leosvelperez marked this pull request as ready for review October 2, 2024 09:36
@leosvelperez leosvelperez merged commit b82ca22 into master Oct 2, 2024
6 checks passed
@leosvelperez leosvelperez deleted the js/fix-library-config-in-package-json branch October 2, 2024 09:40
Copy link

github-actions bot commented Oct 8, 2024

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 8, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants