Skip to content

Conversation

@MIchaelMainer
Copy link
Contributor

A similar PR may already be submitted! Please search among the Pull request before creating one.

Thanks for submitting a pull request! Please provide enough information so that others can review your pull request.

NOTE: PR's will be accepted only in case of appropriate information is provided below

Summary

  • All dev and main branches must have policy
  • Require status checks to pass before merging
  • Require code owner approvals of all changes in main/master and dev branches
  • Require re-approval after new commits
  • Require CodeQL pass

Motivation

Better quality checkins through more process rigor.

Test plan

After commit, validate that PRs can pass all checks.

* All dev and main branches must have policy
* Require status checks to pass before merging
* Require code owner approvals of all changes in main/master and dev branches
* Require re-approval after new commits
@MIchaelMainer MIchaelMainer requested a review from a team as a code owner August 30, 2023 20:28
@MIchaelMainer MIchaelMainer merged commit 8cf3eb6 into dev Aug 31, 2023
@MIchaelMainer MIchaelMainer deleted the mmainer/updatepolicy branch August 31, 2023 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants