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(repo): switch to yarn pnp #326

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

meeroslav
Copy link
Contributor

No description provided.

@meeroslav meeroslav self-assigned this Aug 18, 2023
@meeroslav meeroslav requested a review from FrozenPandaz August 18, 2023 15:18
@nx-cloud
Copy link

nx-cloud bot commented Aug 18, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit fa3d467. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


🟥 Failed Commands
nx affected --target=build --parallel --max-parallel=3
✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@meeroslav meeroslav force-pushed the switch-to-pnp branch 3 times, most recently from b1e3808 to 5d1c943 Compare April 5, 2024 09:23
Copy link

nx-cloud bot commented Apr 5, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit a563ca7. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


🟥 Failed Commands
nx affected --target=e2e --parallel --max-parallel=1 --exclude=nx-ignore-e2e
✅ Successfully ran 3 targets

Sent with 💌 from NxCloud.

@meeroslav
Copy link
Contributor Author

The E2E test fails while running npm install in tmp project

1981 error command sh -c node ./bin/post-install
1982 error /Users/miro/Dev/nrwl/nx-labs/.pnp.cjs:24098
1982 error       Error.captureStackTrace(firstError);
1982 error             ^
1982 error
1982 error Error: Your application tried to access jsonc-parser, but it isn't declared in your dependencies; this makes the require call ambiguous and unsound.
1982 error
1982 error Required package: jsonc-parser
1982 error Required by: /Users/miro/Dev/nrwl/nx-labs/tmp/nx-e2e/proj/node_modules/nx/src/utils/
1982 error
1982 error Require stack:
1982 error - /Users/miro/Dev/nrwl/nx-labs/tmp/nx-e2e/proj/node_modules/nx/src/utils/json.js
1982 error - /Users/miro/Dev/nrwl/nx-labs/tmp/nx-e2e/proj/node_modules/nx/src/utils/fileutils.js
1982 error - /Users/miro/Dev/nrwl/nx-labs/tmp/nx-e2e/proj/node_modules/nx/src/utils/cache-directory.js
1982 error - /Users/miro/Dev/nrwl/nx-labs/tmp/nx-e2e/proj/node_modules/nx/src/project-graph/nx-deps-cache.js
1982 error - /Users/miro/Dev/nrwl/nx-labs/tmp/nx-e2e/proj/node_modules/nx/src/project-graph/project-graph.js
1982 error - /Users/miro/Dev/nrwl/nx-labs/tmp/nx-e2e/proj/node_modules/nx/bin/post-install.js
1982 error     at require$$0.Module._resolveFilename (/Users/miro/Dev/nrwl/nx-labs/.pnp.cjs:24098:13)
1982 error �[90m    at Module._load (node:internal/modules/cjs/loader:985:27)�[39m
1982 error     at require$$0.Module._load (/Users/miro/Dev/nrwl/nx-labs/.pnp.cjs:23989:31)
1982 error �[90m    at Module.require (node:internal/modules/cjs/loader:1235:19)�[39m
1982 error �[90m    at require (node:internal/modules/helpers:176:18)�[39m
1982 error     at Object.<anonymous> (/Users/miro/Dev/nrwl/nx-labs/tmp/nx-e2e/proj/node_modules/�[4mnx�[24m/src/utils/json.js:4:24)
1982 error �[90m    at Module._compile (node:internal/modules/cjs/loader:1376:14)�[39m
1982 error �[90m    at Module._extensions..js (node:internal/modules/cjs/loader:1435:10)�[39m
1982 error     at require$$0.Module._extensions..js (/Users/miro/Dev/nrwl/nx-labs/.pnp.cjs:24141:33)
1982 error �[90m    at Module.load (node:internal/modules/cjs/loader:1207:32)�[39m

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