# Testing PRs/bugs and ability to add labels, reviewers in GitHub

**URL:** <https://gramps.discourse.group/t/testing-prs-bugs-and-ability-to-add-labels-reviewers-in-github/2892>\
**Category:** Development\
**Created:** [October 19, 2022, 1:05am UTC](https://gramps.discourse.group/t/testing-prs-bugs-and-ability-to-add-labels-reviewers-in-github/2892 "2022-10-19T01:05:18Z")\
**Posts on this page:** 6\
**Page:** 1

<div class="post-metadata">

**Author:** ![codefarmer](https://avatars.discourse-cdn.com/v4/letter/c/b9e5f3/32.png) [@codefarmer](https://gramps.discourse.group/u/codefarmer)\
**Post date:** [October 19, 2022, 1:05am UTC](https://gramps.discourse.group/t/testing-prs-bugs-and-ability-to-add-labels-reviewers-in-github/2892/1 "2022-10-19T01:05:19Z")

</div>

In [this post](https://gramps.discourse.group/t/is-there-a-way-i-can-help-some/2873/2) @Nick-Hall mentioned labels like ‘Needs testing’. Using labels is a really nice way for anyone wanting to help to quickly scan PRs and pick issues to contribute their time to.

As a code contributor, I would like to be able to add a label ‘Needs testing’ to a [PR I created](https://github.com/gramps-project/gramps/pull/1395) because it’s been tested only on Windows, and I would hate for it to be merged to the codebase without testing on Linux and MacOS.

In addition to adding labels, it would be great if reviewers could be requested for GitHub PRs based on previous authors of the code. What’s the normal way to request this? Put a comment in the PR?

Thanks.

---

<div class="post-metadata">

**Author:** ![Mattkmmr](https://yyz2.discourse-cdn.com/free1/user_avatar/gramps.discourse.group/mattkmmr/32/410_2.png) [@Mattkmmr](https://gramps.discourse.group/u/Mattkmmr)\
**Post date:** [October 19, 2022, 7:28am UTC](https://gramps.discourse.group/t/testing-prs-bugs-and-ability-to-add-labels-reviewers-in-github/2892/2 "2022-10-19T07:28:34Z")

</div>

> [@codefarmer](#):
>
> it would be great if reviewers could be requested for GitHub PRs based on previous authors of the code.

I think github has the feature to request a code review from a selected person.

---

<div class="post-metadata">

**Author:** ![emyoulation](https://yyz2.discourse-cdn.com/free1/user_avatar/gramps.discourse.group/emyoulation/32/67_2.png) [@emyoulation](https://gramps.discourse.group/u/emyoulation)\
**Post date:** [October 19, 2022, 12:04pm UTC](https://gramps.discourse.group/t/testing-prs-bugs-and-ability-to-add-labels-reviewers-in-github/2892/4 "2022-10-19T12:04:40Z")

</div>

There’s been good response here when asking for Mac & Linux testing to compliment my test on Windoze.

---

<div class="post-metadata">

**Author:** ![Nick-Hall](https://yyz2.discourse-cdn.com/free1/user_avatar/gramps.discourse.group/nick-hall/32/95_2.png) [@Nick-Hall](https://gramps.discourse.group/u/Nick-Hall)\
**Post date:** [October 19, 2022, 1:53pm UTC](https://gramps.discourse.group/t/testing-prs-bugs-and-ability-to-add-labels-reviewers-in-github/2892/5 "2022-10-19T13:53:12Z")

</div>

> [@codefarmer](#):
>
> Using labels is a really nice way for anyone wanting to help to quickly scan PRs and pick issues to contribute their time to.

We can create some new labels if they would be helpful. At the moment “Needs testing” is defined as “A pull request that has been approved but needs testing before it can be merged.”, but I can always create a separate “Approved” label.

Users need to be granted the “Triage” role to be able to add and remove them.

---

<div class="post-metadata">

**Author:** ![codefarmer](https://avatars.discourse-cdn.com/v4/letter/c/b9e5f3/32.png) [@codefarmer](https://gramps.discourse.group/u/codefarmer)\
**Post date:** [October 20, 2022, 10:25pm UTC](https://gramps.discourse.group/t/testing-prs-bugs-and-ability-to-add-labels-reviewers-in-github/2892/6 "2022-10-20T22:25:04Z")

</div>

> [@Nick-Hall](#):
>
> Users need to be granted the “Triage” role to be able to add and remove them.

That’s the missing piece. As @Mattkmmr said, labels and reviewers are a long standing GitHub feature but I wasn’t able to set a label or ask for a review because of the missing Triage role. May I request that role? Thanks.

@emyoulation I will post here when testing is needed for a PR. Thanks!

---

<div class="post-metadata">

**Author:** ![system](https://global.discourse-cdn.com/free1/uploads/gramps/original/1X/2ac1e712b7adf612e31ca08a55419f4cf37c0158.png) [@system](https://gramps.discourse.group/u/system)\
**Post date:** [November 19, 2022, 10:25pm UTC](https://gramps.discourse.group/t/testing-prs-bugs-and-ability-to-add-labels-reviewers-in-github/2892/7 "2022-11-19T22:25:25Z")

</div>

Closed on November 19, 2022, 10:25pm UTC

This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.
