search

natcap / invest

InVEST®: models that map and value the goods and services from nature that sustain and fulfill human life.
Apache License 2.0
166 stars 68 forks source link

Original PR requester should be the reviewer and assignee on PR into release branches #119

Closed phargogh closed 4 years ago

phargogh commented 4 years ago

When a PR is merged into master, our automation creates a new PR from master --> release/X for each release branch in the repo. The approver assigned is the person who merged the PR into master.

Instead, the PR from master --> release/X should be assigned to the person who submitted the PR into master originally. The reason for this is twofold:

  1. The submitter has context for the change (which is especially useful when there's a merge conflict)
  2. It's less work for the reviewer of the original PR
phargogh commented 4 years ago

Once https://github.com/natcap/pygeoprocessing/pull/89 is resolved, see about including this here in this repo.