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

build(docs-infra): update Angular framework/Components/CLI to latest versions #37898

Closed
wants to merge 5 commits into from

Conversation

gkalpak
Copy link
Member

@gkalpak gkalpak commented Jul 2, 2020

The PR updates Angular framework to v10.0.2, Angular Components to v10.0.1 and Angular CLI to v10.0.1. See individual commits for more details.

Fixes #37699.

@gkalpak gkalpak added area: build & ci Related the build and CI infrastructure of the project comp: docs-infra state: WIP target: patch This PR is targeted for the next patch release labels Jul 2, 2020
@ngbot ngbot bot modified the milestone: needsTriage Jul 2, 2020
@mary-poppins
Copy link

You can preview 1ec42dc at https://pr37898-1ec42dc.ngbuilds.io/.

@mary-poppins
Copy link

You can preview 38f1f24 at https://pr37898-38f1f24.ngbuilds.io/.

@gkalpak gkalpak force-pushed the build-aio-update-angular-deps branch from 38f1f24 to 4641ee5 Compare July 2, 2020 20:56
@gkalpak gkalpak marked this pull request as ready for review July 2, 2020 20:58
@mary-poppins
Copy link

You can preview 4641ee5 at https://pr37898-4641ee5.ngbuilds.io/.

@@ -3,7 +3,7 @@
"master": {
"uncompressed": {
"runtime-es2015": 2987,
"main-es2015": 453213,
"main-es2015": 450880,
Copy link
Member

Choose a reason for hiding this comment

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

woohoo! 2.5k decrease?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes 🎉 (Part of it is the framework and part of it is Components.)

@gkalpak gkalpak added action: merge The PR is ready for merge by the caretaker and removed state: WIP labels Jul 6, 2020
Copy link
Member

@pkozlowski-opensource pkozlowski-opensource left a comment

Choose a reason for hiding this comment

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

LGTM

Reviewed for: size-tracking

Copy link
Member

@pkozlowski-opensource pkozlowski-opensource left a comment

Choose a reason for hiding this comment

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

Reviewed-for: size-tracking

Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

LGTM

Reviewed-for: size-tracking

@@ -6,21 +6,27 @@ describe('CustomIconRegistry', () => {
const mockHttp: any = {};
const mockSanitizer: any = {};
const mockDocument: any = {};
const mockErrorHandler: any = {};
Copy link
Member

Choose a reason for hiding this comment

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

I wouldn't block submission on this, but I would do

const fakeErrorHandler = {
  handleError: e => { console.error(e); },
};

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated.
(Also updated the other names from mock* to fake* (since they are more fakes than mocks - although arguably they are neither 😁).

const registry = new CustomIconRegistry(mockHttp, mockSanitizer, mockDocument, svgIcons);

const registry =
new CustomIconRegistry(mockHttp, mockSanitizer, mockDocument, mockErrorHandler, svgIcons);
Copy link
Member

Choose a reason for hiding this comment

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

FYI there's actually an "official" FakeMatIconRegistry in @angular/material/icon/testing now

Copy link
Member Author

Choose a reason for hiding this comment

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

👍
For future reference, we are testing our custom MatIconRegistry here, so using FakeMatIconRegistry is not applicable.

This commit updates the version of Angular framework used in angular.io
to version 10.0.2. It also features a commit message with a 100+ chars
long body.
This commit updates the version of Angular Components used in angular.io
to version 10.0.1. It also updates the angular.io app to adapt to
breaking changes.
This commit removes some duplicate imports of Material themes and
theming-related utilities. While this change does not have any impact on
the size of the generated `styles.css` file, it silences a build warning
pointing to [Avoiding duplicated theming styles][1].

[1]: https://github.com/angular/components/blob/db4b0cd1bff0bbb1741d22975bcfe1069aea3a53/guides/duplicate-theming-styles.md
This commit updates the version of Angular CLI used in angular.io to
version 10.0.1. It also reverts some changes (namely commits 38dfbc7
and eee2fd2) which were made due to an older bug that is fixed in
the latest version. See angular#37688 for more details.

Fixes angular#37699
@gkalpak gkalpak force-pushed the build-aio-update-angular-deps branch from 4641ee5 to f1c1872 Compare July 7, 2020 11:27
@gkalpak gkalpak force-pushed the build-aio-update-angular-deps branch from f1c1872 to 574620f Compare July 7, 2020 11:34
@mary-poppins
Copy link

You can preview 574620f at https://pr37898-574620f.ngbuilds.io/.

@jelbourn jelbourn removed the request for review from IgorMinar July 8, 2020 02:17
atscott pushed a commit that referenced this pull request Jul 8, 2020
This commit updates the version of Angular framework used in angular.io
to version 10.0.2. It also features a commit message with a 100+ chars
long body.

PR Close #37898
atscott pushed a commit that referenced this pull request Jul 8, 2020
This commit updates the version of Angular Components used in angular.io
to version 10.0.1. It also updates the angular.io app to adapt to
breaking changes.

PR Close #37898
atscott pushed a commit that referenced this pull request Jul 8, 2020
This commit removes some duplicate imports of Material themes and
theming-related utilities. While this change does not have any impact on
the size of the generated `styles.css` file, it silences a build warning
pointing to [Avoiding duplicated theming styles][1].

[1]: https://github.com/angular/components/blob/db4b0cd1bff0bbb1741d22975bcfe1069aea3a53/guides/duplicate-theming-styles.md

PR Close #37898
atscott pushed a commit that referenced this pull request Jul 8, 2020
This commit updates the version of Angular CLI used in angular.io to
version 10.0.1. It also reverts some changes (namely commits 38dfbc7
and eee2fd2) which were made due to an older bug that is fixed in
the latest version. See #37688 for more details.

Fixes #37699

PR Close #37898
@atscott atscott closed this in 54373cc Jul 8, 2020
atscott pushed a commit that referenced this pull request Jul 8, 2020
This commit updates the version of Angular Components used in angular.io
to version 10.0.1. It also updates the angular.io app to adapt to
breaking changes.

PR Close #37898
atscott pushed a commit that referenced this pull request Jul 8, 2020
This commit removes some duplicate imports of Material themes and
theming-related utilities. While this change does not have any impact on
the size of the generated `styles.css` file, it silences a build warning
pointing to [Avoiding duplicated theming styles][1].

[1]: https://github.com/angular/components/blob/db4b0cd1bff0bbb1741d22975bcfe1069aea3a53/guides/duplicate-theming-styles.md

PR Close #37898
atscott pushed a commit that referenced this pull request Jul 8, 2020
This commit updates the version of Angular CLI used in angular.io to
version 10.0.1. It also reverts some changes (namely commits 38dfbc7
and eee2fd2) which were made due to an older bug that is fixed in
the latest version. See #37688 for more details.

Fixes #37699

PR Close #37898
@gkalpak gkalpak deleted the build-aio-update-angular-deps branch July 9, 2020 09:01
@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 Aug 8, 2020
profanis pushed a commit to profanis/angular that referenced this pull request Sep 5, 2020
This commit updates the version of Angular framework used in angular.io
to version 10.0.2. It also features a commit message with a 100+ chars
long body.

PR Close angular#37898
profanis pushed a commit to profanis/angular that referenced this pull request Sep 5, 2020
This commit updates the version of Angular Components used in angular.io
to version 10.0.1. It also updates the angular.io app to adapt to
breaking changes.

PR Close angular#37898
profanis pushed a commit to profanis/angular that referenced this pull request Sep 5, 2020
…ar#37898)

This commit removes some duplicate imports of Material themes and
theming-related utilities. While this change does not have any impact on
the size of the generated `styles.css` file, it silences a build warning
pointing to [Avoiding duplicated theming styles][1].

[1]: https://github.com/angular/components/blob/db4b0cd1bff0bbb1741d22975bcfe1069aea3a53/guides/duplicate-theming-styles.md

PR Close angular#37898
profanis pushed a commit to profanis/angular that referenced this pull request Sep 5, 2020
This commit updates the version of Angular CLI used in angular.io to
version 10.0.1. It also reverts some changes (namely commits 38dfbc7
and eee2fd2) which were made due to an older bug that is fixed in
the latest version. See angular#37688 for more details.

Fixes angular#37699

PR Close angular#37898
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 area: build & ci Related the build and CI infrastructure of the project cla: yes target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flaky test_aio CircleCI job due to size check
7 participants