Page MenuHomePhabricator

[web] Replace {||} with {} in identity-structs.cjs.flow
ClosedPublic

Authored by ashoat on Nov 28 2023, 8:40 PM.
Tags
None
Referenced Files
F2898514: D10088.id33975.diff
Sat, Oct 5, 3:29 AM
Unknown Object (File)
Tue, Oct 1, 11:30 PM
Unknown Object (File)
Tue, Oct 1, 11:30 PM
Unknown Object (File)
Tue, Oct 1, 11:30 PM
Unknown Object (File)
Tue, Oct 1, 11:30 PM
Unknown Object (File)
Tue, Oct 1, 11:30 PM
Unknown Object (File)
Thu, Sep 5, 5:07 PM
Unknown Object (File)
Sep 4 2024, 11:49 PM
Subscribers

Details

Summary

Following the Flow upgrade, I was experimenting with the various codemods in flow-upgrade. I gave removeExplicitlyExactObjectTypeSyntax a try, since we have exact_by_default=true set in our .flowconfigs, and as such {} is an alias to {||}.

The codemod found a couple locations in this file. For the sake of consistency, I replaced them with {}, like we do elsewhere in our codebase.

Depends on D10087

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage