-
Notifications
You must be signed in to change notification settings - Fork 12k
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
test(@ngtools/webpack): add source-map-support. #7191
Conversation
karma plugin relies on source-map-support being a dependency of karma-source-map-support and npm 3+ flat dependency tree. pnpm/pnpm#863
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.
Can you also change the commit name to fix(@angular/cli): add explicit source-map-support dependency
?
Commit messages affect the changelogs, and this change is for @angular/cli
proper.
@@ -91,6 +91,7 @@ | |||
"silent-error": "^1.0.0", | |||
"source-map": "^0.5.6", | |||
"source-map-loader": "^0.2.0", | |||
"source-map-support": "^0.4.1", |
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.
Can you add this to https://github.com/angular/angular-cli/blob/master/packages/%40angular/cli/package.json as well?
We have several packages being deployed from the same repo so multiple package.json need to be updated.
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.
Dependency added and also fixed the enumerate in karma.ts.
I agree with the conclusions in pnpm/pnpm#863 and would like to see this merged. |
Done, thanks! |
karma plugin relies on source-map-support being a dependency of karma-source-map-support and npm 3+ flat dependency tree. pnpm/pnpm#863
karma plugin relies on source-map-support being a dependency of karma-source-map-support and npm 3+ flat dependency tree. pnpm/pnpm#863
…r#7191) karma plugin relies on source-map-support being a dependency of karma-source-map-support and npm 3+ flat dependency tree. pnpm/pnpm#863
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. |
karma plugin relies on source-map-support being a dependency of karma-source-map-support and npm 3+ flat dependency tree.
pnpm/pnpm#863