Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

  • Yes, but only someone with write access can perform the merge. You may need to remind the reviewer who gives the approval that they will need to perform the merge.

    • Note that this step is time sensitive. If a PR sits around too long before merging (for weeks or, in the case of repos with lots of activity, even just a few days) it may need to be rebased again, increasing effort for both the author and – sometimes at least – their reviewers.

External Contributions GitHub Board

...

https://github.com/orgs/openedx/projects/19/views/1  

  • PRs not authored by 2U or tCRIL engineers go to this board when they come in.

  • Once added to the board, they need to get reviewed and merged.

  • Use the board’s columns to change the status of a PR (you can also change it on the right side of the PR page itself). The following guidelines should help determine appropriate status (as well as labels to add to the PR to capture additional information): Contributions Board Status & Label Breakdown.

Core Contributors and their repos

...

Core Contributors to the Open edX Project  

  • CCs on a repo can provide review and formal approval for a PR and merge it.

  • Many repos do not have CCs or have so few that they may not be able to handle all incoming OSPRs. In this case, you would need to get 2U (or possibly tCRIL) reviewers to approve and merge PRs.

Backstage

...

https://backstage.openedx.org/catalog?filters%5Bkind%5D=component&filters%5Buser%5D=all  

  • This tool allows you to see details about various GitHub repositories, including who the maintainers are. You can use this information to ping maintainers on PRs.

  • NOTE! This is still a pilot and only a handful of repos are represented.

...