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
Unknown Object (File)
Thu, Mar 13, 5:13 PM
Unknown Object (File)
Mon, Mar 10, 4:21 PM
Unknown Object (File)
Mon, Mar 10, 4:21 PM
Unknown Object (File)
Mon, Mar 10, 4:21 PM
Unknown Object (File)
Mon, Mar 10, 3:48 PM
Unknown Object (File)
Mon, Mar 10, 3:47 PM
Unknown Object (File)
Tue, Mar 4, 12:28 AM
Unknown Object (File)
Tue, Feb 25, 2:02 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
Lint Not Applicable
Unit
Tests Not Applicable