-
Notifications
You must be signed in to change notification settings - Fork 2.6k
docs(nx-dev): document playwright output merging w/ atomizer #32830
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
Conversation
View your CI Pipeline Execution ↗ for commit 6d8e39d
☁️ Nx Cloud last updated this comment at |
The latest updates on your projects. Learn more about Vercel for GitHub.
|
...-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc
Outdated
Show resolved
Hide resolved
...-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc
Show resolved
Hide resolved
...-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc
Outdated
Show resolved
Hide resolved
Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
cb802b0
to
dbfe884
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nx Cloud is proposing a fix for your failed CI:
We've fixed the broken internal links in the new Playwright documentation that were causing the build to fail. The links now correctly point to /ci/features/split-e2e-tasks
instead of the non-existent /docs/features/ci-features/split-e2e-tasks
path.
We verified this fix by re-running nx-dev:build
.
Suggested Fix changes
diff --git a/astro-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc b/astro-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc
index 5182ac5871..9af32c3cb9 100644
--- a/astro-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc
+++ b/astro-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc
@@ -5,7 +5,7 @@ sidebar:
filter: 'type:Guides'
---
-When running Playwright tests in CI environments, Nx can [split tests across multiple parallel tasks](/docs/features/ci-features/split-e2e-tasks) for faster execution.
+When running Playwright tests in CI environments, Nx can [split tests across multiple parallel tasks](/ci/features/split-e2e-tasks) for faster execution.
However, this creates separate test reports for each task.
With the Nx Plugin, the Playwright report merging feature allows you to combine these individual reports into a single, unified report.
@@ -19,7 +19,7 @@ With the Nx Plugin, the Playwright report merging feature allows you to combine
- Nx workspace with `@nx/playwright` plugin
- Playwright tests configured in your project
-- CI environment with [task atomization enabled](/docs/features/ci-features/split-e2e-tasks)
+- CI environment with [task atomization enabled](/ci/features/split-e2e-tasks)
- Nx v21.6.1
{% aside type="note" title="Nx Version"%}
diff --git a/docs/shared/packages/playwright/merge-atomized-outputs.md b/docs/shared/packages/playwright/merge-atomized-outputs.md
index 4e2b3fa9d5..37d700e4be 100644
--- a/docs/shared/packages/playwright/merge-atomized-outputs.md
+++ b/docs/shared/packages/playwright/merge-atomized-outputs.md
@@ -5,7 +5,7 @@ description: Learn how to enable output merging for atomized Playwright tests
# Merge Atomized Playwright Outputs
-When running Playwright tests in CI environments, Nx can [split tests across multiple parallel tasks](/docs/features/ci-features/split-e2e-tasks) for faster execution.
+When running Playwright tests in CI environments, Nx can [split tests across multiple parallel tasks](/ci/features/split-e2e-tasks) for faster execution.
However, this creates separate test reports for each task.
With the Nx Plugin, the Playwright report merging feature allows you to combine these individual reports into a single, unified report.
@@ -19,7 +19,7 @@ With the Nx Plugin, the Playwright report merging feature allows you to combine
- Nx workspace with `@nx/playwright` plugin
- Playwright tests configured in your project
-- CI environment with [task atomization enabled](/docs/features/ci-features/split-e2e-tasks)
+- CI environment with [task atomization enabled](/ci/features/split-e2e-tasks)
- Nx v21.6.1
{% callout type="note" title="Nx Version"%}
⚙️ An Nx Cloud workspace admin can disable these reviews in workspace settings.
...-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc
Outdated
Show resolved
Hide resolved
...-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc
Outdated
Show resolved
Hide resolved
...-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc
Outdated
Show resolved
Hide resolved
...-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc
Outdated
Show resolved
Hide resolved
dbfe884
to
f87fb7f
Compare
...-docs/src/content/docs/technologies/test-tools/playwright/Guides/merge-atomized-outputs.mdoc
Outdated
Show resolved
Hide resolved
f87fb7f
to
00fdc0a
Compare
00fdc0a
to
6d8e39d
Compare
fixes: DOC-208