CDCgov / IDWA

Intelligent Data Workflow Automation
Apache License 2.0
1 stars 1 forks source link

Decide as a cross team on branching strategies #29

Closed derekadombek closed 5 months ago

derekadombek commented 5 months ago

User Story

Currently anybody on our team is allowed to push to main. As everybody on the team starts contributing, this is going to cause conflicts. We should use PR's and block merging to main without approvals for this reason. We need to get together and discuss how we want to approve merges to main, do we want a standard mainline approach? How many approvals do PR's need, 1 or 2? Do we agree to automatically squash during merge?

Acceptance Criteria

We should at least require the use of PR's and block merges to main without approval.