Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: add CODEOWNERS; edx/community-engineering #23

Merged
merged 1 commit into from
May 26, 2021
Merged

Commits on May 26, 2021

  1. build: add CODEOWNERS; edx/community-engineering

    Background
    ==========
    As part of our Squad-based ownership, we should stay on top of what
    happens in these repositories. However, due to the number of
    repositories (and subsequently pull requests) across the edX ecosystem,
    it is challenging to stay on top of notifications, separating the
    'signal' from the 'noise'. Email filters can go a long way to taming
    Inbox notifications, but this is manual and requires maintenance as
    Squad ownership changes. It also fails to account for Github-specific behavior.
    
    Proposal
    ========
    By leveraging Github support for `CODEOWNERS` files [1],
    we can ensure that our team is at least CCed explicitly, here,
    in the form a requested review. This request is just that, a request,
    not a requirement; we are not enacting any new merge requirements
    at this time.
    
    - [1] https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners
    stvstnfrd committed May 26, 2021
    Configuration menu
    Copy the full SHA
    7a1194f View commit details
    Browse the repository at this point in the history