Skip to content

feat(@angular/cli): add flag to control chunk naming #7007

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

Merged
merged 1 commit into from
Jul 19, 2017

Conversation

filipesilva
Copy link
Contributor

@filipesilva filipesilva commented Jul 17, 2017

Followup to #6881

Allow controlling chunk naming via the --named-chunks flag, which can be set on .angular-cli.json as well.

Defaults to true in development, false in production.

Fix #6999

hansl
hansl previously approved these changes Jul 17, 2017
@@ -77,6 +77,15 @@
- *service*: Options for generating a service.
- *flat* (`boolean`): Flag to indicate if a dir is created. Default is `true`.
- *spec* (`boolean`): Specifies if a spec file is generated. Default is `true`.
- *build*: Properties to be passed to the serve command
Copy link
Contributor

Choose a reason for hiding this comment

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

typo:
Properties to be passed to the serve build command

- *build*: Properties to be passed to the serve command
- *sourcemaps* (`boolean`): Output sourcemaps.
- *baseHref* (`string`): Base url for the application being built.
- *progress* (`boolean`): The ssl key used by the server. Default is `true`.
Copy link
Contributor

Choose a reason for hiding this comment

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

wrong description?

@filipesilva
Copy link
Contributor Author

Good catches @antonybudianto, thanks!

Copy link
Contributor

@Brocco Brocco left a comment

Choose a reason for hiding this comment

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

One minor nit.

<details>
<summary>named-chunks</summary>
<p>
<code>--named-chunks</code> (aliases: <code>-scd</code>)
Copy link
Contributor

Choose a reason for hiding this comment

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

The alias scd does not match the one in code nc.

@filipesilva filipesilva force-pushed the named-chunks-flag branch 2 times, most recently from a8ac917 to ceeddfe Compare July 19, 2017 15:37
Followup to angular#6881

Allow controlling chunk naming via the `--named-chunks` flag, which can be set on `.angular-cli.json` as well.

Defaults to true in development, false in production.
@Brocco Brocco merged commit 826c634 into angular:master Jul 19, 2017
@filipesilva filipesilva deleted the named-chunks-flag branch July 19, 2017 18:17
@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 Sep 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(@angular/cli): specify naming convention for lazy chunks
5 participants