Skip to content

PR Labels

Jump to bottom
Bill Sacks edited this page Sep 21, 2018 · 4 revisions

Pull Requests (PRs) can be given a number of different labels. All PRs should be given a status label. Other labels are optional. Some issue labels can also apply to PRs.

status

This label defines whether the PR is ready for final review, testing and merge to master.

  • status: work in progress: The author has submitted this for an initial review, but is still working on it.

  • status: later: The CTSM administrators / gatekeepers are not ready to bring in this PR yet. For example, it might change answers at a time when we don't want answer changes, or it may have some dependencies on other changes.

  • status: ready: This PR doesn't fall into any of the above categories, and so is ready for final review, testing and merge to master. Note that a PR can have the ready label even if reviewers have requested changes: a PR should not come to master until it has the ready label and reviewers have given their approval and all necessary tests are passing.

Clone this wiki locally