Upgrade Angular from 9.0.1 to 12 #91
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Updated Angular from 9.1.13 to 10.2.5 - Updated Angular CLI from 9.1.15 to 10.2.4 - Updated TypeScript from 3.7.5 to 4.0.8 - Fixed deprecated RxJS imports (rxjs/Observable -> rxjs, rxjs/Subscription -> rxjs) - Updated tslint to version 6 - Renamed browserslist to .browserslistrc - Updated tslib to version 2 Co-Authored-By: Kevin Tang <kevin.tang513@gmail.com>
- Updated Angular from 10.2.5 to 11.2.14 - Updated Angular CLI from 10.2.4 to 11.2.19 - Updated TypeScript from 4.0.8 to 4.1.6 - Removed deprecated extractCss option from angular.json - Updated relativeLinkResolution in RouterModule to use 'legacy' explicitly - Updated various testing and build dependencies Co-Authored-By: Kevin Tang <kevin.tang513@gmail.com>
- Updated Angular from 11.2.14 to 12.2.17 - Updated Angular CLI from 11.2.19 to 12.2.18 - Updated TypeScript from 4.1.6 to 4.3.5 - Updated zone.js from 0.10.3 to 0.11.8 - Ivy is now mandatory (enabled by default) - Webpack 5 is now used by default - Removed emitDecoratorMetadata TypeScript compiler option - Updated zone.js imports in polyfills - Replaced deprecated '--prod' flag in package.json scripts - Various migrations for Angular 12 compatibility Co-Authored-By: Kevin Tang <kevin.tang513@gmail.com>
All deprecated RxJS imports have been fixed to use modern import paths. The rxjs-compat compatibility package is no longer needed. Co-Authored-By: Kevin Tang <kevin.tang513@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Upgrade Angular from 9.0.1 to 12
Summary
This PR upgrades the Angular application from version 9.0.1 to 12.2.17 through three sequential major version upgrades (9→10→11→12). This is a substantial upgrade that brings the application up to date with improved Ivy rendering engine, Webpack 5, and better tree-shaking.
Key Changes:
extractCssandaotoptions, Ivy now mandatoryrelativeLinkResolution: 'legacy'to preserve old behavior (should be updated to 'corrected' in future)Files Modified:
user.component.ts,item-details.component.ts, andhackernews-api.service.tsangular.jsonwith Angular 12 build configurationpackage.jsonandyarn.lockwith new dependency versionspolyfills.tsandsrc/test.tsReview & Testing Checklist for Human
This is a high-risk change due to the scope of the upgrade. The build succeeds, but comprehensive end-to-end testing is required:
npm run build && npm run precache && npm run static-serve)npm run buildand verify bundle sizes are acceptable (currently ~2.33 MB ES2015, ~3.24 MB ES5)Notes
angular-10-stable,angular-11-stable,angular-12-stablefor easy rollback if neededrelativeLinkResolution: 'legacy'to maintain old routing behavior - consider updating to'corrected'in a future PRLink to Devin run: https://app.devin.ai/sessions/0132ad5569d24294b30cc6a4e424ffd7
Requested by: Kevin Tang (@kevintang513)