Page MenuHomePhabricator

[lib][native][web] Make Flow types for Redux state read-only
ClosedPublic

Authored by ashoat on Jul 31 2023, 11:33 AM.
Tags
None
Referenced Files
F3386232: D8685.id29302.diff
Fri, Nov 29, 3:57 AM
F3386226: D8685.id29427.diff
Fri, Nov 29, 3:54 AM
F3385694: D8685.diff
Fri, Nov 29, 1:47 AM
Unknown Object (File)
Sat, Nov 23, 11:39 PM
Unknown Object (File)
Sat, Nov 23, 4:29 AM
Unknown Object (File)
Sat, Nov 23, 4:28 AM
Unknown Object (File)
Fri, Nov 1, 7:39 AM
Unknown Object (File)
Fri, Nov 1, 7:39 AM
Subscribers

Details

Summary

This probably should've been done a long time ago. Best practice is to modernize a Flow type whenever you touch it, but evidently people were too intimidated to try and modernize BaseAppState and its various related types.

I went ahead and did this. It looks like it went well, with no new Flow errors.

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage