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

docs(testing): add example of using Playwright's --project argument with Nx #29107

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

Coly010
Copy link
Contributor

@Coly010 Coly010 commented Nov 28, 2024

Current Behavior

Missing documentation on how to use Playwright's --project argument with Nx.
Because Nx also has a --project argument, passing it to the command like nx e2e app --project=firefox will result in the --project being stripped from the command that is sent to Playwright.

Expected Behavior

The fix is simple enough, change the command to be nx e2e app -- --project=firefox to ensure the argument is forwarded correctly to Playwright.
Add some information to the Playwright plugin's overview documentation to explain this.

Related Issue(s)

Fixes #26965

@Coly010 Coly010 requested review from a team as code owners November 28, 2024 11:57
@Coly010 Coly010 requested a review from jaysoo November 28, 2024 11:57
@Coly010 Coly010 self-assigned this Nov 28, 2024
Copy link

vercel bot commented Nov 28, 2024

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

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Nov 28, 2024 0:08am

@Coly010 Coly010 changed the title docs(playwright): add example of using Playwright's argument with Nx docs(testing): add example of using Playwright's --project argument with Nx Nov 28, 2024
Copy link
Member

@juristr juristr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@Coly010 Coly010 merged commit c66b99c into master Nov 28, 2024
6 checks passed
@Coly010 Coly010 deleted the playwright/docs-to-show-filter-project branch November 28, 2024 13:37
FrozenPandaz pushed a commit that referenced this pull request Nov 28, 2024
…ith Nx (#29107)

## Current Behavior
Missing documentation on how to use Playwright's `--project` argument
with Nx.
Because Nx also has a `--project` argument, passing it to the command
like `nx e2e app --project=firefox` will result in the `--project` being
stripped from the command that is sent to Playwright.

## Expected Behavior
The fix is simple enough, change the command to be `nx e2e app --
--project=firefox` to ensure the argument is forwarded correctly to
Playwright.
Add some information to the Playwright plugin's overview documentation
to explain this.

## Related Issue(s)

Fixes #26965

(cherry picked from commit c66b99c)
Copy link

github-actions bot commented Dec 5, 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 Dec 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@nx/playwright ignoring --project argument
2 participants