Skip to content

feat(js): ignore scoped export conditions #32063

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aidant
Copy link
Contributor

@aidant aidant commented Jul 24, 2025

There is an emerging pattern of using scoped custom export conditions for pointing to source. See:

Current Behavior

When including an export condition that points to the source code the typescript plugin does not add a build target. However, if this export condition is named development the build target is added.

Expected Behavior

Scoped export conditions pointing at source should not remove the build target.

Related Issue(s)

@aidant aidant requested a review from a team as a code owner July 24, 2025 02:06
@aidant aidant requested a review from ndcunningham July 24, 2025 02:06
Copy link

vercel bot commented Jul 24, 2025

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

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Jul 24, 2025 4:27am

Copy link
Contributor

nx-cloud bot commented Jul 24, 2025

View your CI Pipeline Execution ↗ for commit 6a70c29

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 49m 14s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 1m 50s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 4s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 4s View ↗
nx documentation ✅ Succeeded 1m 57s View ↗

☁️ Nx Cloud last updated this comment at 2025-07-24 05:11:24 UTC

@aidant aidant force-pushed the feat-ignore-scoped-export-conditions branch from 14da10d to 6a70c29 Compare July 24, 2025 04:15
@aidant aidant changed the title feat(ts): ignore scoped export conditions feat(js): ignore scoped export conditions Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant