Page MenuHomePhabricator

[keyserver][lib] handleAsyncPromise -> ignorePromiseRejections
ClosedPublic

Authored by ashoat on Dec 4 2023, 1:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 7:02 PM
Unknown Object (File)
Tue, Nov 26, 10:42 AM
Unknown Object (File)
Mon, Nov 25, 3:53 PM
Unknown Object (File)
Sat, Nov 23, 4:32 PM
Unknown Object (File)
Nov 22 2024, 2:41 AM
Unknown Object (File)
Nov 19 2024, 9:31 PM
Unknown Object (File)
Nov 19 2024, 7:27 PM
Unknown Object (File)
Nov 8 2024, 6:20 AM
Subscribers

Details

Summary

As part of enabling the unused-promise Flow lint (ENG-5907), I want to make two changes to this function:

  1. Renaming it to ignorePromiseRejections. I think this captures what it does much better.
  2. Move it to lib. The Lint exposed some issues in lib utilities that we use from keyserver, and to fix those we'll want access to ignorePromiseRejections from lib.
Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable