Page MenuHomePhabricator

[lib] Add ridiculous branching to `delete-account-spec` to appease `flow`
ClosedPublic

Authored by atul on Dec 8 2023, 11:58 AM.
Tags
None
Referenced Files
F3158480: D10270.id34602.diff
Tue, Nov 5, 9:56 PM
F3157150: D10270.diff
Tue, Nov 5, 7:27 PM
Unknown Object (File)
Tue, Oct 15, 9:51 PM
Unknown Object (File)
Tue, Oct 15, 9:51 PM
Unknown Object (File)
Tue, Oct 15, 9:51 PM
Unknown Object (File)
Tue, Oct 15, 9:51 PM
Unknown Object (File)
Tue, Oct 15, 9:51 PM
Unknown Object (File)
Sep 17 2024, 7:15 AM
Subscribers
None

Details

Summary

I'm completely aware that the amount of branching and repetition here is ridiculous. This is just to help us get towards 0 flow issues, at which point I will definitely come back and do noop refactors to tidy these up.

NOTE: CI is expected to fail. I got to a point in my local environment where all flow issues were resolved, but based on earlier fedback in the stack I'm breaking them down so things are easier to review.. even if that means that intermediate steps fail.

[skip-ci]


Depends on D10268

Test Plan

Flow/CI/etc.

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 24913
Build 216813: arc lint + arc unit