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.

Sign up for GitHub

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

FED-3249 Update docs about migrator companion codemod #959

Merged
merged 3 commits into from
Oct 24, 2024
Merged

FED-3249 Update docs about migrator companion codemod #959

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
baacf1e
Update note about migrator companion codemod
sydneyjodon-wk Oct 15, 2024
d47eb79
Add note about connect prop codemod
sydneyjodon-wk Oct 23, 2024
7a6eec8
Address feedback
sydneyjodon-wk Oct 24, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion doc/null_safety/null_safe_migration.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,9 +51,11 @@ dart pub global run over_react_codemod:null_safety_migrator_companion --yes-to-a
```

This codemod will:
- Add nullability hints to props/state that are defaulted/initialized in class components.
greglittlefield-wf marked this conversation as resolved.
Show resolved Hide resolved
- Add nullability hints to state mixin/class fields
- These hints will cause defaulted/initialized values to be migrated as "late required".
greglittlefield-wf marked this conversation as resolved.
Show resolved Hide resolved
See our [prop requiredness and nullability](#prop-requiredness-and-nullability) docs for more details on whether you should keep them required following the migration.
- All non-initialized state fields will have optional nullable hints.
- Add nullable hints for callback ref types.

#### Required props codemod

Expand Down