Skip to content
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(core): allow customizing the location of the plugin e2e project #30390

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

Conversation

tinesoft
Copy link
Contributor

Current Behavior

When generating a new plugin with @nx/plugin and with e2e tests, the e2e project is located in the same folder as the plugin project itself.

This is fine when the plugin project is located at workspace root, but feels "weird" when the plugin is located in a sub directory (for e.g inside a packages/my-plugin folder). People may want to group e2e project inside a e2e folder.

Expected Behavior

Allow customizing where to put the -e2e project, and default to the same folder as the plugin

Related Issue(s)

Fixes #

@tinesoft tinesoft requested review from a team as code owners March 16, 2025 09:31
@tinesoft tinesoft requested a review from JamesHenry March 16, 2025 09:31
Copy link

vercel bot commented Mar 16, 2025

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

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Mar 21, 2025 8:30am

Copy link

nx-cloud bot commented Mar 16, 2025

View your CI Pipeline Execution ↗ for commit a805072.

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

☁️ Nx Cloud last updated this comment at 2025-03-21 08:39:47 UTC

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