Skip to content

Angular upgrade from version 16 to 17#7085

Draft
cdavalos7 wants to merge 2 commits intotensorflow:masterfrom
cdavalos7:feature/upgrade16-17
Draft

Angular upgrade from version 16 to 17#7085
cdavalos7 wants to merge 2 commits intotensorflow:masterfrom
cdavalos7:feature/upgrade16-17

Conversation

@cdavalos7
Copy link
Copy Markdown
Contributor

Motivation for features / changes

This PR is the second step in a planned major Angular upgrade cycle, where each major version will be delivered in a separate PR, incrementally progressing until the project reaches Angular 21. This specific PR upgrades TensorBoard from Angular 16 to Angular 1

Technical description of changes

  • Upgrade Angular, NgRx, and Material from v16 to v17
    • Upgrade TypeScript to 5.2.2 and zone.js to 0.14
    • Use Node.js 18 in Bazel and CI (required by Angular 17)
    • Update build-tooling patch to match Angular 17 version
    • Fix Bazel TypeScript compiler crash caused by TypeScript 5.x
    • Fix 3 snackbar tests that broke due to Material 17 behavior change
    • Remove deprecated Material legacy theme APIs

Screenshots of UI changes (or N/A)

Detailed steps to verify changes work correctly (as executed by you)

  • All 2022 webapp tests pass
  • yarn lint passes with Node 18
  • App builds and runs locally

Alternate designs / implementations considered (or N/A)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant