Skip to content

Comments

refactor(deps): migrate @angular/forms from v15.2.9 to v21.1.5#7059

Open
digital-wizard48 wants to merge 1 commit intotensorflow:masterfrom
digital-wizard48:deps/upgrade--angular-forms-21.1.5
Open

refactor(deps): migrate @angular/forms from v15.2.9 to v21.1.5#7059
digital-wizard48 wants to merge 1 commit intotensorflow:masterfrom
digital-wizard48:deps/upgrade--angular-forms-21.1.5

Conversation

@digital-wizard48
Copy link

Changes

Version Bump

  • Updated @angular/forms from ^15.2.9 to ^21.1.5 in package.json

Source Code Migration

settings_dialog_component.ts

  • Removed deprecated as FormControl type assertions on FormControl instances
  • Angular Forms v14+ introduced strictly-typed reactive forms (FormControl<T>), making the old as FormControl cast pattern unnecessary and discouraged
  • reloadPeriodControl and paginationControl now use the typed FormControl<number> generic form, which provides better type safety and is the recommended pattern in Angular 14+

Notes

  • The changelog for this release states "No user facing changes" — the primary migration work here is adopting the typed reactive forms pattern that was introduced in Angular 14 and is the standard in Angular 21
  • All existing FormsModule, ReactiveFormsModule, Validators, and AbstractControl usages remain compatible and require no changes
  • The ValidatorFn return type {[key: string]: any} | null in createIntegerValidator() continues to be valid

This PR was auto-generated by Gittensor upgrade bot using Claude AI

@google-cla
Copy link

google-cla bot commented Feb 21, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@digital-wizard48
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

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.

2 participants