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

Upgrade to angular 13. #16961

Merged
merged 50 commits into from
Dec 26, 2021
Merged

Upgrade to angular 13. #16961

merged 50 commits into from
Dec 26, 2021

Conversation

mshima
Copy link
Member

@mshima mshima commented Nov 9, 2021

Angular 13 generated packages are esm now.

  • Jest mock cannot mock an esm package, so switch to an alternative.
  • Switch dayjs to use esm exports. Requires adjusts to tsconfig and jest.config.

Note: generated project is commonjs.
ESM support is done by tooling (typescript, angular compiler, jest, webpack)

Closes #16960.


Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

@mshima mshima changed the title Migrate to angular 13. Upgrade to angular 13. Nov 9, 2021
@github-actions github-actions bot added theme: angular theme: dependencies Pull requests that update a dependency file theme: front labels Nov 9, 2021
@mshima

This comment has been minimized.

@mshima

This comment has been minimized.

@DanielFran
Copy link
Member

@mshima https://github.com/ng-bootstrap/ng-bootstrap/releases/tag/11.0.0 has been released

@mshima mshima mentioned this pull request Dec 17, 2021
6 tasks
@mshima mshima marked this pull request as ready for review December 18, 2021 11:09
@mshima
Copy link
Member Author

mshima commented Dec 25, 2021

PR is ready.
Reviews are welcome.

jest.spyOn(service, 'save').mockReturnValue(of({}));
<%_ if (enableTranslation) { _%>
mockLanguageService.currentLang = '<%= nativeLanguage %>';
mockTranslateService.currentLang = '<%= nativeLanguage %>';
Copy link
Contributor

Choose a reason for hiding this comment

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

Not a mock anymore, right?

@Tcharl
Copy link
Contributor

Tcharl commented Dec 26, 2021

Everything LGTM: feel free to merge

@DanielFran DanielFran merged commit 69a39e8 into main Dec 26, 2021
@DanielFran DanielFran deleted the skip_ci-angular13 branch December 26, 2021 14:13
@pascalgrimaud pascalgrimaud added this to the 7.5.0 milestone Jan 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to Angular 13
5 participants