Home GnuPG

Rename "getReviewerStatus()" to "getReviewers()"
0ceab7d36f91Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Rename "getReviewerStatus()" to "getReviewers()"

Summary:
Ref T10967. Improves some method names:

  • Revision->getReviewerStatus() -> Revision->getReviewers()
  • Revision->attachReviewerStatus() -> Revision->attachReviewers()
  • Reviewer->getStatus() -> Reviewer->getReviewerStatus() (this is mostly to make this more greppable)

Test Plan:

  • bunch o' grep
  • Browsed around.
  • If I missed anything, it should fatal in an obvious way. We have a lot of other getStatus() calls and it's hard to be sure I got them all.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10967

Differential Revision: https://secure.phabricator.com/D17522

Details

Provenance
epriestley <git@epriestley.com>Authored on Mar 20 2017, 11:04 PM
Parents
rPHABa15df4f8d580: Rename "needReviewerStatus()" into "needReviewers()"
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHAB0ceab7d36f91: Rename "getReviewerStatus()" to "getReviewers()" (authored by epriestley <git@epriestley.com>).Mar 21 2017, 1:11 AM