Page MenuHomePhabricator

[desktop] Enable unused-promise=error Flow lint
ClosedPublic

Authored by ashoat on Dec 5 2023, 7:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jan 29, 1:49 AM
Unknown Object (File)
Jan 17 2025, 3:11 AM
Unknown Object (File)
Jan 17 2025, 12:05 AM
Unknown Object (File)
Jan 12 2025, 2:35 AM
Unknown Object (File)
Dec 24 2024, 11:12 AM
Unknown Object (File)
Dec 24 2024, 11:12 AM
Unknown Object (File)
Dec 24 2024, 11:12 AM
Unknown Object (File)
Dec 24 2024, 11:12 AM
Subscribers

Details

Summary

Following up on what I said in D10184:

Once I get an answer there, I'll decide on how to proceed for desktop, and submit some follow-up diffs to enable unused-promise for it as well.

I received a partial answer, and did some follow-up research myself. It looks like we can treat desktop like native and web, since it doesn't crash on unhandled promise rejections, unlike keyserver.

As such, this diff mostly prefix invocations with void, with one exception where I add an await instead in an async IIFE where the result is thrown away anyways.

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/postflow
Lint
No Lint Coverage
Unit
No Test Coverage