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
docs: describe how to configure CommonJS modules #37331
Conversation
You can preview d480de2 at https://pr37331-d480de2.ngbuilds.io/. |
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.
Feedback incorporated. Thank you!
You can preview aa6f505 at https://pr37331-aa6f505.ngbuilds.io/. |
You can preview 9a8e59c at https://pr37331-9a8e59c.ngbuilds.io/. |
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.
Thanks for the feedback!
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.
2 small nits
- Can we changes
{@a commonjs }
to{@a commonjs-dependencies }
- Can you fixup your commits please?
You can preview d6e7954 at https://pr37331-d6e7954.ngbuilds.io/. |
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.
LGTM after the suggested tweaks, but I would also consider adding language around our intention to turn off the commonjs support in the future and making it an opt-in instead.
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.
you can checkout https://angular.io/guide/docs-style-guide#markdown-and-html if you have any doubts
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.
Adding more suggestions
You can preview 12db3ac at https://pr37331-12db3ac.ngbuilds.io/. |
In version 10, we have a new option for the `angular.json` file, `allowedCommonJsDependencies`, so users can opt in to support CommonJS modules.
In version 10, we have a new option for the `angular.json` file, `allowedCommonJsDependencies`, so users can opt in to support CommonJS modules. PR Close #37331
In version 10, we have a new option for the `angular.json` file, `allowedCommonJsDependencies`, so users can opt in to support CommonJS modules. PR Close angular#37331
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. |
In version 10, we have a new option for the `angular.json` file, `allowedCommonJsDependencies`, so users can opt in to support CommonJS modules. PR Close angular#37331
In version 10, we have a new option for the
angular.json
file,allowedCommonJsDependencies
, so users can opt in to supportCommonJS modules.
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Current documentation does not describe how to configure the
angular.json
file to support CommonJS modules.Issue Number: N/A
What is the new behavior?
Add section that describes how to configure the
angular.json
file to support CommonJS modules.Does this PR introduce a breaking change?
Other information