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

Angular: Add Angular 11.1 support #13704

Merged
merged 2 commits into from
Jan 22, 2021
Merged

Angular: Add Angular 11.1 support #13704

merged 2 commits into from
Jan 22, 2021

Conversation

gaetanmaisse
Copy link
Member

Issue: #13689, #13563 surely #13693

What I did

Some internal plumbing about webpack config changed in the latest version of Angular, for details see angular/angular-cli@eb30a92. As Storybook is using private function of @angular-devkit/build-angular I added the new required properties in the webpack config.

I also updated the Angular example and the Angular dependencies in the app to Angular 11.1.

How to test

  • As Angular example has been updated to Angular 11.1 the CI will check if everything is OK

@gaetanmaisse gaetanmaisse requested review from ThibaudAV, shilman and a team January 22, 2021 12:19
@github-actions
Copy link
Contributor

Fails
🚫

PR is not labeled with one of: ["cleanup","BREAKING CHANGE","feature request","bug","documentation","maintenance","dependencies","other"]

Generated by 🚫 dangerJS against 2997357

@gaetanmaisse gaetanmaisse added angular maintenance User-facing maintenance tasks labels Jan 22, 2021
@shilman shilman changed the title Angular: add Angular 11 support Angular: Add Angular 11 support Jan 22, 2021
@shilman shilman added this to the 6.1.x milestone Jan 22, 2021
@shilman shilman added the run e2e extended test suite Run the e2e extended test suite in CircleCI workflows label Jan 22, 2021
@gaetanmaisse gaetanmaisse changed the title Angular: Add Angular 11 support Angular: Add Angular 11.1 support Jan 22, 2021
Copy link
Member

@shilman shilman 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 jumping on top of this @gaetanmaisse! Smoke test failures are unrelated to this PR and @j3rem1e and I are investigating separately. Merging so we can test this & patch to master if it's working

@shilman shilman merged commit 0c21c46 into next Jan 22, 2021
@shilman shilman deleted the feat-add-angular-11-support branch January 22, 2021 13:13
@shilman shilman added patch:yes Bugfix & documentation PR that need to be picked to main branch patch:done Patch/release PRs already cherry-picked to main/release branch labels Jan 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
angular maintenance User-facing maintenance tasks patch:done Patch/release PRs already cherry-picked to main/release branch patch:yes Bugfix & documentation PR that need to be picked to main branch run e2e extended test suite Run the e2e extended test suite in CircleCI workflows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants