When closing an issue in the PullRequest merge dialog, make the issue transitioner to be the PullRequest author instead of the merger. (BB-6909)

Hugo Visser avatarHugo Visser created an issue

We use pull requests as a form of code review; each team member works in a fork and uses pull request to get their code reviewed and included in the master repository. They work on issues that are in the master repo's issue tracker. When a team member references or closes an issue in the commit and that commit is accepted in a pull request, the issue is then updated by the user accepting the pull request.

It would be better in my opinion to have the issue update either by the team member issuing the pull request if that account is a team member on the repo to which the pull request is sent. If this is not the case the issue can be updated as it is now, by the account that accepts the pull request.

Comments (3)

  1. Log in to comment
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.