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
Unknown Object (File)
Mon, Feb 17, 12:37 AM
Unknown Object (File)
Sun, Feb 16, 11:28 PM
Unknown Object (File)
Sun, Feb 9, 12:08 AM
Unknown Object (File)
Thu, Jan 30, 12:19 AM
Unknown Object (File)
Wed, Jan 22, 4:04 PM
Unknown Object (File)
Jan 17 2025, 10:06 AM
Unknown Object (File)
Jan 16 2025, 4:05 PM
Unknown Object (File)
Jan 13 2025, 11:55 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