Paths

Table of Contentst

Differential D4335

[lib] Rename `threadIsWatched` to `isThreadWatched`
ClosedPublic
Actions

Authored by atul on Jun 22 2022, 10:16 PM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

• jacek
tomek
• abosh

Commits

rCOMM79a5ee1c5c9c: [lib] Rename `threadIsWatched` to `isThreadWatched`

Summary

I think this is a better name.

This is an "opinionated" change, so open to abandoning if others disagree.

Depends on D4334

Test Plan

flow

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Jun 22 2022, 10:16 PM

Herald added subscribers: • adrian, ashoat. · View Herald TranscriptJun 22 2022, 10:16 PM

Harbormaster completed remote builds in B9898: Diff 13690.Jun 22 2022, 10:21 PM

atul requested review of this revision.Jun 22 2022, 10:21 PM

atul edited the summary of this revision. (Show Details)Jun 22 2022, 10:27 PM

atul added a parent revision: D4334: [lib] Rename `threadsToMessageIDsFromMessageInfos` to `mapThreadsToMessageIDsFromOrderedMessageInfos`.

atul added a child revision: D4336: [lib] Simplify logic in `mapThreadsToMessageIDsFromOrderedMessageInfos`.Jun 22 2022, 10:31 PM

tomek accepted this revision.Jun 23 2022, 10:59 AM

This revision is now accepted and ready to land.Jun 23 2022, 10:59 AM

Agree with this. threadIsWatched-type boolean names feel like they imply a value of true, since it sounds like "The thread is watched" versus "Is the thread watched?"

cherrypick before landing

This revision was landed with ongoing or failed builds.Jun 24 2022, 5:42 PM

Closed by commit rCOMM79a5ee1c5c9c: [lib] Rename `threadIsWatched` to `isThreadWatched`. · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM79a5ee1c5c9c: [lib] Rename `threadIsWatched` to `isThreadWatched`.

Harbormaster completed remote builds in B9959: Diff 13772.Jun 24 2022, 5:46 PM

Revision Contents
Changeset List

Path

Size

lib/

reducers/

message-reducer.js

10 lines

Diff 13773

lib/reducers/message-reducer.js

Loading...