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

Update CODEOWNERS to include engineers from both product groups for critical path files #19183

Open
4 tasks
lukeheath opened this issue May 21, 2024 · 0 comments
Open
4 tasks
Assignees
Labels
#g-endpoint-ops Endpoint ops product group #g-mdm MDM product group :release Ready to write code. Scheduled in a release. See "Making changes" in handbook. story A user story defining an entire feature

Comments

@lukeheath
Copy link
Member

Goal

User story
As a Fleet contributor,
I want to know which files require special consideration or review from other engineers
so that I can make sure not to merge code that could break other workflows.

Context

Changes

Engineering

  • Define critical files that require a special review process.
  • Define engineers from each product group that will be assigned reviewers.
  • Submit PR with changes to codeowners file to implement review requirement.
  • Document the special review process in the Engineering handbook.
@lukeheath lukeheath added story A user story defining an entire feature :release Ready to write code. Scheduled in a release. See "Making changes" in handbook. #g-mdm MDM product group #g-endpoint-ops Endpoint ops product group labels May 21, 2024
@lukeheath lukeheath added this to the 4.51.0-tentative milestone May 22, 2024
@lukeheath lukeheath self-assigned this Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#g-endpoint-ops Endpoint ops product group #g-mdm MDM product group :release Ready to write code. Scheduled in a release. See "Making changes" in handbook. story A user story defining an entire feature
Development

No branches or pull requests

3 participants