-
Notifications
You must be signed in to change notification settings - Fork 27.1k
Closed
Labels
P4A relatively minor issue that is not relevant to core functionsA relatively minor issue that is not relevant to core functionsarea: coreIssues related to the framework runtimeIssues related to the framework runtimefreq2: mediumtype: bug/fix
Milestone
Description
I'm submitting a...
[x] Other... Please describe: Tracking issue for internal cleanup.
What needs to be done
In order to quickly turn on strictPropertyInitialization flag in the whole code base, we introduced ! on every non-initialized class field. Each one of those fields needs to be individually checked and decided whether:
- the field should be initialized. Thus
foo!: string, should becomefoo = 'default'. - the field should be marked as optional. Thus
foo!: string, should becomefoo?: string. - the field is reliably initialized, but TS cannot infer that (initialization can occur outside the ctor, for example), so
!should be kept.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P4A relatively minor issue that is not relevant to core functionsA relatively minor issue that is not relevant to core functionsarea: coreIssues related to the framework runtimeIssues related to the framework runtimefreq2: mediumtype: bug/fix