Skip to content

Commit

Permalink
[web] Replace {||} with {} in identity-structs.cjs.flow
Browse files Browse the repository at this point in the history
Summary:
Following the Flow upgrade, I was experimenting with the various [codemods in](https://www.npmjs.com/package/flow-upgrade) `flow-upgrade`. I gave `removeExplicitlyExactObjectTypeSyntax` a try, since we have `exact_by_default=true` set in our `.flowconfig`s, 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

Reviewers: varun, bartek

Reviewed By: bartek

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D10088
  • Loading branch information
Ashoat committed Nov 29, 2023
1 parent 3d12522 commit b92c233
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions web/protobufs/identity-structs.cjs.flow
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ import {

declare export class Empty extends Message {
serializeBinary(): Uint8Array;
toObject(includeInstance?: boolean): {||};
static toObject(includeInstance: boolean, msg: Empty): {||};
toObject(includeInstance?: boolean): {};
static toObject(includeInstance: boolean, msg: Empty): {};
static serializeBinaryToWriter(message: Empty, writer: BinaryWriter): void;
static deserializeBinary(bytes: Uint8Array): Empty;
static deserializeBinaryFromReader(message: Empty, reader: BinaryReader): Empty;
Expand Down

0 comments on commit b92c233

Please sign in to comment.