API changes should be reviewed by experts
Problem
Changes to our APIs lack oversight. This risks:
- Inconsistent design.
- Breaking changes being introduced outside of policy.
- Performance and security issues.
Proposal
Merge requests that introduce changes to the API should be reviewed by domain experts to ensure:
- Consistency and quality in design.
- Oversight of performance and security.
- Breaking changes follow policy.
GraphQL:
- Make more items alpha and formalise this process
👉 #393119 (comment 1325391616) - Review guidelines #393119 (comment 1325389763)
- GraphQL trainee programme? #393119 (comment 1325955239)
- Have Danger suggest a GraphQL review if changes to those files are present #393119 (comment 1324935429)
Edited by charlie ablett