Page MenuHomePhabricator

[lib] Return `mergeNewMessages(...)` directly where possible in `reduceMessageStore(...)`
ClosedPublic

Authored by atul on Jun 22 2022, 3:50 PM.
Tags
None
Referenced Files
F2060033: D4338.id13776.diff
Thu, Jun 20, 10:48 PM
Unknown Object (File)
Wed, Jun 19, 6:47 PM
Unknown Object (File)
Sat, Jun 15, 4:50 PM
Unknown Object (File)
Thu, Jun 13, 8:12 AM
Unknown Object (File)
Wed, Jun 12, 9:49 AM
Unknown Object (File)
Mon, Jun 3, 6:18 PM
Unknown Object (File)
May 18 2024, 8:08 PM
Unknown Object (File)
May 1 2024, 1:29 AM
Subscribers

Details

Summary

Basically D4337, but for all the other mergeNewMessages(...) callsites.

This is just a noop refactor, trying to simplify message-reducer


Depends on D4337

Test Plan

Close reading, flow

Diff Detail

Repository
rCOMM Comm
Branch
june21 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul requested review of this revision.Jun 22 2022, 3:55 PM
This revision is now accepted and ready to land.Jun 23 2022, 4:52 AM

rebase after cherrypicking and before landing

This revision was landed with ongoing or failed builds.Jun 24 2022, 10:47 AM
This revision was automatically updated to reflect the committed changes.