...
Continue to remain in compliance with edX’s security (PCI, etc), privacy (GDPR, etc), and a11y commitments.
Since edX.org deploys continuously off of master, please work with your champion to determine when appropriate merge windows are; guidelines will vary repo-to-repo. Guidelines generally are here: Merge Timing Guidelines for Public Repositories
Do not regress in the forward movement we are concurrently making on technical ownership. For each Committer contribution, we will need to ensure:
edX Product accepts future maintenance costs of new/enhanced features in the platform.
edX Engineering owning team acknowledges acceptance of ownership of changes.
Support Committers who are at varying levels of technical expertise, though they must demonstrate excellent judgment and accountability.
Be transparent and open about the existence and eligibility of the Committer program.
Constrain Committer’s relationship to the Open edX (public) codebase, without exposing edX-confidential business or data.
Hold an equal bar for both edX engineers and community engineers, in the long-term. In the future, for instance, edX engineers might earn merge rights just as other contributors to the platform.
Respect edX’s right to modify and terminate the Committer program.
...
Right | Responsibility | |
Legal | Be an individual Committer regardless of organization affiliation | Sign individual (and, if needed, entity) contributor agreements, per edX legal requirements |
Renew annual continued interest in being an individual Committer | Complete annual training for security, privacy, a11y, and architecture | |
Merge | Merge to master, using defensive CI/CD techniques such as feature toggles | Ensure approved review process is upheld and be on-call to address issues with recently merged PRs. Follow Merge Timing Guidelines for Public Repositories |
(for Pilot) 2 designated edx-internal champions to support Committer with on-call issues | Retain strong communication with the champions, including visibility into expected merges and time availabilities | |
Own | Review PRs and suggest technical changes in designated repositories | Learn and advocate for clean code, quality, and architecture principles and practices (per repository’s definition of done) |
Co-establish technical direction of designated repositories1 | Documenting and reviewing decisions in ADRs (and OEPs) and maintaining READMEs, HowTos, etc | |
Co-maintain prioritized backlog of needed technical improvements of designated repositories. | Negotiate and allocate a regular percentage of time toward technical upkeep, including refactorings and other items listed in this column. | |
(in Future) Co-own designated repositories | Ongoing upgrade and feature maintenance and other ownership costs of designated repositories | |
Process | Vote in the selection process for future Committers | Train new/young developers2 to help hone their own judgment and skills, while following our code of conduct |
Co-establish the selection process for Committers | Lead by example |
...
The Committer program is currently in its Phase 2, as described in Core Contributor Committers: Phases for Rollout. The program relies on individual edX champions to support designated Committers; as of Sept 2021, we are piloting a team-based model with some newly added Committers.
...