-
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(aio): upgrade CLI version to cope with new Angular 6.0.0-beta.1 release #21772
build(aio): upgrade CLI version to cope with new Angular 6.0.0-beta.1 release #21772
Conversation
9af4a2b
to
cec3707
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.
LGTM (as long as Travis is happy 😃)
You can preview cec3707 at https://pr21772-cec3707.ngbuilds.io/. |
You can preview 569f718 at https://pr21772-569f718.ngbuilds.io/. |
e789c6c
to
5d8b7c9
Compare
@@ -177,8 +177,8 @@ to a component's `@Component` decorator: | |||
|
|||
<code-tabs> | |||
<code-pane title="src/app/hero-app.component.ts (CSS in file)" path="component-styles/src/app/hero-app.component.1.ts"></code-pane> | |||
<code-pane title="src/app/hero-app.component.css" path="component-styles/src/app/hero-app.component.1.css"></code-pane> | |||
</code-tabs> | |||
<code-pane title="src/app/hero-app.component.css" path="component-styles/src/app/hero-app.component.css"></code-pane> |
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.
This is the only significant change in this file.
The rest is just trailing whitespace removal
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.
Why rename it? AFAICT, the file wan intentionally suffixed .1.css
, so that it would not be included n stackblitz/zip.
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.
Because there is a component that references this file (without the 1) and breaks the build if the file is not there.
It is a trade off we could rename it in that component but then the 1 would leak into the docs.
… release Before version 1.6 of Angular CLI there was a check that prevented use of Angular compiler CLI with major version 6.
The AOT compiler needs relative paths so that it can find the imported stylesheets.
The code in the example was referring to `hero-app.component.css` but this did not exist.
5d8b7c9
to
0e7b6d5
Compare
You can preview 0e7b6d5 at https://pr21772-0e7b6d5.ngbuilds.io/. |
The AOT compiler needs relative paths so that it can find the imported stylesheets. PR Close #21772
The code in the example was referring to `hero-app.component.css` but this did not exist. PR Close #21772
The AOT compiler needs relative paths so that it can find the imported stylesheets. PR Close #21772
The code in the example was referring to `hero-app.component.css` but this did not exist. PR Close #21772
… release (angular#21772) Before version 1.6 of Angular CLI there was a check that prevented use of Angular compiler CLI with major version 6. PR Close angular#21772
The AOT compiler needs relative paths so that it can find the imported stylesheets. PR Close angular#21772
…r#21772) The code in the example was referring to `hero-app.component.css` but this did not exist. PR Close angular#21772
… release (angular#21772) Before version 1.6 of Angular CLI there was a check that prevented use of Angular compiler CLI with major version 6. PR Close angular#21772
The AOT compiler needs relative paths so that it can find the imported stylesheets. PR Close angular#21772
…r#21772) The code in the example was referring to `hero-app.component.css` but this did not exist. PR Close angular#21772
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. |
This should un-break master.
Before version 1.6 of Angular CLI there was a check that prevented use of Angular
compiler CLI with major version 6.