[lib] Rename `reduceThreadUpdates` => `generateOpsForThreadUpdates`
ClosedPublic
Actions

Authored by atul on Jun 2 2022, 10:41 PM.

Details

Reviewers

• jacek
tomek

Commits

rCOMM49e3bbd53c0f: [lib] Rename `reduceThreadUpdates` => `generateOpsForThreadUpdates`

Summary

More accurately reflects the behavior of the function

Depends on D4197

Test Plan

No other occurences of "reduceThreadUpdates" in the codebase

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Jun 2 2022, 10:41 PM

Herald added subscribers: • abosh, • adrian, ashoat. · View Herald TranscriptJun 2 2022, 10:41 PM

Harbormaster completed remote builds in B9603: Diff 13313.Jun 2 2022, 10:52 PM

atul requested review of this revision.Jun 2 2022, 10:52 PM

atul added a child revision: D4200: [lib] Destructure some properties in `reduceThreadInfos` to improve readability.Jun 2 2022, 10:58 PM

tomek accepted this revision.Jun 3 2022, 9:58 AM

This revision is now accepted and ready to land.Jun 3 2022, 9:58 AM

cherrypick before landing

This revision was landed with ongoing or failed builds.Jun 8 2022, 12:35 PM

Closed by commit rCOMM49e3bbd53c0f: [lib] Rename `reduceThreadUpdates` => `generateOpsForThreadUpdates`. · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM49e3bbd53c0f: [lib] Rename `reduceThreadUpdates` => `generateOpsForThreadUpdates`.

Harbormaster completed remote builds in B9689: Diff 13428.Jun 8 2022, 12:40 PM

Revision Contents
Changeset List

Path

Size

lib/

reducers/

thread-reducer.js

4 lines

Diff 13429

View Options

lib/reducers/thread-reducer.js

[lib] Rename `reduceThreadUpdates` => `generateOpsForThreadUpdates`ClosedPublicActions