[native] Don't make Crash.onPressWipe an async function
ClosedPublic
Actions

Authored by ashoat on Dec 4 2023, 1:15 PM.

Details

Reviewers

bartek
michal

Commits

rCOMM27f414488c8e: [native] Don't make Crash.onPressWipe an async function

Summary

While working on the unused-promise Flow lint, I noticed that there is no await keyword here, so there's no point to it being an async function.

Depends on D10169

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Dec 4 2023, 1:15 PM

Herald added a subscriber: tomek. · View Herald TranscriptDec 4 2023, 1:15 PM

ashoat added a child revision: D10171: [native] await sendMultimediaMessage in KeyboardInputHost.onMediaGalleryItemSelected.Dec 4 2023, 1:15 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Dec 4 2023, 1:42 PM

Harbormaster failed remote builds in B24749: Diff 34212!

Rebase

ashoat published this revision for review.Dec 4 2023, 6:27 PM

Harbormaster completed remote builds in B24766: Diff 34229.Dec 4 2023, 6:52 PM

michal accepted this revision.Dec 5 2023, 2:55 AM

This revision is now accepted and ready to land.Dec 5 2023, 2:55 AM

Closed by commit rCOMM27f414488c8e: [native] Don't make Crash.onPressWipe an async function (authored by ashoat). · Explain WhyDec 5 2023, 12:32 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM27f414488c8e: [native] Don't make Crash.onPressWipe an async function.

Revision Contents
Changeset List

Path

Size

native/

crash.react.js

2 lines

Diff 34212

View Options

[native] Don't make Crash.onPressWipe an async functionClosedPublicActions