Page MenuHomePhabricator

Rename "needReviewerStatus()" into "needReviewers()"
ClosedPublic

Authored by epriestley on Mar 20 2017, 9:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 12:36 AM
Unknown Object (File)
Thu, Apr 11, 8:30 AM
Unknown Object (File)
Thu, Apr 11, 8:30 AM
Unknown Object (File)
Mon, Apr 8, 5:41 PM
Unknown Object (File)
Mon, Apr 8, 12:29 AM
Unknown Object (File)
Sun, Apr 7, 12:29 AM
Unknown Object (File)
Sat, Apr 6, 2:27 AM
Unknown Object (File)
Mar 21 2024, 1:29 PM
Subscribers
None

Details

Summary

Ref T10967. The old name was because we had a getReviewers() tied to needRelationships(), rename this method to use a simpler and more clear name.

Test Plan

grep, browsed around.

Diff Detail

Repository
rP Phabricator
Branch
rev8
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16045
Build 21280: Run Core Tests
Build 21279: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Mar 20 2017, 11:44 PM
This revision was automatically updated to reflect the committed changes.