-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
NX experiment #33220
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
NX experiment #33220
Changes from all commits
250127b
ec6d354
3b0a2d7
7733316
69fbe0c
fbc2af4
ac61afb
ec1bd5d
81a8497
69106f4
f35970d
d80cf75
c31e808
ae1ebf8
f04dd28
d58d659
4446340
5dd20f9
7c89c17
6e6a31c
02e22fa
40cdf8b
39a4409
b4b8b24
729408e
b9a60d1
c7244fa
73841fd
a4f5eda
7aa225e
c289d52
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,7 +6,7 @@ steps: | |
| - git-shallow-clone/checkout_advanced: | ||
| clone_options: '--depth 1 --verbose' | ||
| - attach_workspace: | ||
| at: . | ||
| at: /tmp | ||
| - run: | ||
| name: Install dependencies | ||
| command: yarn install --no-immutable | ||
|
|
@@ -18,8 +18,8 @@ steps: | |
| command: yarn playwright-e2e | ||
| working_directory: test-storybooks/portable-stories-kitchen-sink/react-vitest-3 | ||
| - store_test_results: | ||
| path: test-results | ||
| path: storybook/test-results | ||
| - store_artifacts: | ||
| path: test-storybooks/portable-stories-kitchen-sink/react-vitest-3/test-results/ | ||
| path: storybook/test-storybooks/portable-stories-kitchen-sink/react-vitest-3/test-results/ | ||
| destination: playwright | ||
|
Comment on lines
20
to
24
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain🏁 Script executed: #!/bin/bash
# Read the full CircleCI config file to understand the working_directory and paths
cat -n ".circleci/src/jobs/e2e-ui-vitest-3.yml"Repository: storybookjs/storybook Length of output: 1046 🏁 Script executed: #!/bin/bash
# Look for similar e2e CircleCI config files to understand the pattern
find .circleci/src/jobs -name "e2e-ui-vitest*.yml" -type f | sortRepository: storybookjs/storybook Length of output: 103 🏁 Script executed: #!/bin/bash
# Check for all CircleCI job files and search for storybook/ references
find .circleci/src/jobs -name "*.yml" -type f | head -20Repository: storybookjs/storybook Length of output: 799 🏁 Script executed: #!/bin/bash
# Check e2e-ui.yml for comparison
cat -n ".circleci/src/jobs/e2e-ui.yml"Repository: storybookjs/storybook Length of output: 1019 🏁 Script executed: #!/bin/bash
# Check build.yml to understand the workspace structure
head -50 ".circleci/src/jobs/build.yml"Repository: storybookjs/storybook Length of output: 1378 🏁 Script executed: #!/bin/bash
# Search for where test-storybooks directories are created
rg -n "test-storybooks" ".circleci/src/jobs/" -A 2 -B 2Repository: storybookjs/storybook Length of output: 5021 🏁 Script executed: #!/bin/bash
# Look for main CircleCI config
find .circleci -name "config.yml" -o -name "config.yaml" | head -5Repository: storybookjs/storybook Length of output: 85 🏁 Script executed: #!/bin/bash
# Search for e2e-ui-vitest-3 in the main config
rg -n "e2e-ui-vitest-3" ".circleci/config.yml" -A 5 -B 5Repository: storybookjs/storybook Length of output: 1609 🏁 Script executed: #!/bin/bash
# Search for where test-storybooks is created or persisted
rg -n "persist_to_workspace" ".circleci/src/jobs/" -A 20 | grep -E "(persist_to_workspace|test-storybooks|portable-stories)" -A 3 -B 1Repository: storybookjs/storybook Length of output: 800 🏁 Script executed: #!/bin/bash
# Search for test-storybooks directory in the repository
find . -maxdepth 2 -type d -name "test-storybooks" 2>/dev/nullRepository: storybookjs/storybook Length of output: 82 🏁 Script executed: #!/antml/bin/bash
# Check what git-shallow-clone does and understand the checkout flow
rg -n "git-shallow-clone" ".circleci/src/" -A 5 -B 2 | head -30Repository: storybookjs/storybook Length of output: 1783 🏁 Script executed: #!/bin/bash
# Let's look for vitest config to see where test results are actually generated
find . -name "vitest.config.*" -type f | grep -i "test-storybooks\|portable-stories" | head -5Repository: storybookjs/storybook Length of output: 47 🏁 Script executed: #!/bin/bash
# Search for vitest configs
find . -name "vitest.config.*" -type fRepository: storybookjs/storybook Length of output: 1922 🏁 Script executed: #!/bin/bash
# Check the structure of test-storybooks
ls -la test-storybooks/ | head -20Repository: storybookjs/storybook Length of output: 571 🏁 Script executed: #!/bin/bash
# Check portable-stories-kitchen-sink structure
ls -la test-storybooks/portable-stories-kitchen-sink/Repository: storybookjs/storybook Length of output: 453 🏁 Script executed: #!/bin/bash
# Check react-vitest-3 structure and package.json
ls -la test-storybooks/portable-stories-kitchen-sink/react-vitest-3/Repository: storybookjs/storybook Length of output: 1174 Working directory paths must include The CircleCI configuration has a critical path mismatch:
The working directory paths are missing the This same issue exists in 🤖 Prompt for AI Agents |
||
| - report-workflow-on-failure | ||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -6,14 +6,14 @@ steps: | |||||||||||||||||||||||||||||||||||
| - git-shallow-clone/checkout_advanced: | ||||||||||||||||||||||||||||||||||||
| clone_options: --depth 1 --verbose | ||||||||||||||||||||||||||||||||||||
| - attach_workspace: | ||||||||||||||||||||||||||||||||||||
| at: . | ||||||||||||||||||||||||||||||||||||
| at: /tmp | ||||||||||||||||||||||||||||||||||||
| - run: | ||||||||||||||||||||||||||||||||||||
| command: | | ||||||||||||||||||||||||||||||||||||
| cd code | ||||||||||||||||||||||||||||||||||||
| TEST_FILES=$(circleci tests glob "**/*.{stories}.{ts,tsx,js,jsx,cjs}" | sed "/^e2e-tests\//d" | sed "/^node_modules\//d") | ||||||||||||||||||||||||||||||||||||
| echo "$TEST_FILES" | circleci tests run --command="xargs yarn test --reporter=junit --reporter=default --outputFile=../test-results/junit-${CIRCLE_NODE_INDEX}.xml" --verbose | ||||||||||||||||||||||||||||||||||||
| name: Run tests | ||||||||||||||||||||||||||||||||||||
| - store_test_results: | ||||||||||||||||||||||||||||||||||||
| path: test-results | ||||||||||||||||||||||||||||||||||||
| path: storybook/test-results | ||||||||||||||||||||||||||||||||||||
|
Comment on lines
10
to
+17
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🚨 Critical: Test results path mismatch between output and store location. Line 14 outputs test results to With the workspace attached at Update the test output path to match the new store location: - echo "$TEST_FILES" | circleci tests run --command="xargs yarn test --reporter=junit --reporter=default --outputFile=../test-results/junit-${CIRCLE_NODE_INDEX}.xml" --verbose
+ echo "$TEST_FILES" | circleci tests run --command="xargs yarn test --reporter=junit --reporter=default --outputFile=../storybook/test-results/junit-${CIRCLE_NODE_INDEX}.xml" --verboseOr, if the output directory structure should be different, update the 📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||
| - report-workflow-on-failure | ||||||||||||||||||||||||||||||||||||
| - cancel-workflow-on-failure | ||||||||||||||||||||||||||||||||||||
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.
cpoptions are in the wrong place and will break sandbox copyingThe
cpinvocation is currently:On GNU
cp, options must come before the first non‑option argument; as written,-r/--remove-destinationare treated as paths, so this command will fail and the sandbox won’t be copied.Recommend:
🤖 Prompt for AI Agents