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

fix(devkit): enhance pkg require errors and rescue a failing generator #30185

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

Conversation

zzeni
Copy link

@zzeni zzeni commented Feb 26, 2025

Please note that this change affects both next and devkit packages. I've decided to attribute it to the one with the broader impact (devkit).

Current Behavior

On a new nx monorepo, when you install @nx/next and run:

yarn nx generate @nx/next:application --directory=test-app --name=test-app --no-interactive --dry-run

the command fails, even though without the dryRun flag it passes. Due to the error, the dryRun output is obscured and there's no obvious reason for it.

Expected Behavior

The proper dryRun output is printed and a warning message informs that playwright will be installed once not in a dryRun mode

Related Issue(s)

Fixes #30184

@zzeni zzeni requested review from a team as code owners February 26, 2025 14:07
@zzeni zzeni requested review from AgentEnder and jaysoo February 26, 2025 14:07
Copy link

vercel bot commented Feb 26, 2025

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 Feb 26, 2025 2:10pm

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.

nx generate @nx/next:application dry-run fails when @nx/playwright is not yet installed as a dependency
1 participant