-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
build(docs-infra): fix playground lezer parsing #55349
Conversation
9f37525
to
40c0082
Compare
Deployed adev-preview for 6971d63 to: https://ng-dev-previews-fw--pr-angular-angular-55349-adev-prev-2aq4m6k6.web.app Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt. |
When we started fixing the version mismatch with local 1st-party packages, we also re-routed all dependencies like `@lezer/javascript` into `adev/node_modules`. This works fine, but results in a different version mismatch because the codemirror dependencies may resolve the Angular version from `/node_modules`- causing some subtle complex runtime error. This commit fixes this by only re-routing dependencies that have dependency on e.g. `@angular/core` into `adev/node_modules`. Fixes angular#55298.
follow up for angular-devkit which does not include peer deps in their depset
bb00580
to
6971d63
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.
Thank you for this !
@devversion One squash away and we're good here ! |
@JeanMeche fixup commits are automatically squashed upon merge 😄 |
This PR was merged into the repository by commit 3471c41. |
When we started fixing the version mismatch with local 1st-party packages, we also re-routed all dependencies like `@lezer/javascript` into `adev/node_modules`. This works fine, but results in a different version mismatch because the codemirror dependencies may resolve the Angular version from `/node_modules`- causing some subtle complex runtime error. This commit fixes this by only re-routing dependencies that have dependency on e.g. `@angular/core` into `adev/node_modules`. Fixes #55298. PR Close #55349
When we started fixing the version mismatch with local 1st-party packages, we also re-routed all dependencies like `@lezer/javascript` into `adev/node_modules`. This works fine, but results in a different version mismatch because the codemirror dependencies may resolve the Angular version from `/node_modules`- causing some subtle complex runtime error. This commit fixes this by only re-routing dependencies that have dependency on e.g. `@angular/core` into `adev/node_modules`. Fixes angular#55298. PR Close angular#55349
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
When we started fixing the version mismatch with local 1st-party packages, we also re-routed all dependencies like
@lezer/javascript
intoadev/node_modules
. This works fine, but results in a different version mismatch because the codemirror dependencies may resolve the Angular version from/node_modules
- causing some subtle complex runtime error.This commit fixes this by only re-routing dependencies that have dependency on e.g.
@angular/core
intoadev/node_modules
.Fixes #55298.