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

Add links to generated configs #17906

Merged
merged 3 commits into from
Jun 15, 2020
Merged

Add links to generated configs #17906

merged 3 commits into from
Jun 15, 2020

Conversation

alan-agius4
Copy link
Collaborator

@alan-agius4 alan-agius4 commented Jun 11, 2020

See individual commits.

Blocked on: angular/angular#37533

We now add a link to generated tsconfig.json to make is easier for users to find relevant information.
We now add a link to generated app package.json to make is easier for users to find relevant information.
@@ -34,7 +34,10 @@ export default function (): Rule {
const pkgJsonContent = {
name: strings.dasherize(projectName),
private: true,
description: `This is a special package.json file that is not used by package managers. It is however used to tell the tools and bundlers whether the code under this directory is free of code with non-local side-effect. Any code that does have non-local side-effects can't be well optimized (tree-shaken) and will result in unnecessary increased payload size. It should be safe to set this option to 'false' for new applications, but existing code bases could be broken when built with the production config if the application code does contain non-local side-effects that the application depends on.`,
description_1: 'This is a special package.json file that is not used by package managers.',
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In VS code, this is unreadable because it will not be wrapped in multilines line in GitHub. Thus, I split it in multiple descriptions.

Copy link
Contributor

@IgorMinar IgorMinar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm. Thanks for the cleanup!

@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Jun 14, 2020
@alan-agius4 alan-agius4 merged commit 88d0eaf into angular:master Jun 15, 2020
@alan-agius4 alan-agius4 deleted the config-links branch June 15, 2020 06:48
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jul 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants