From ebef0439a9a06107d2f151280620f6cf2da09b22 Mon Sep 17 00:00:00 2001 From: Vincent Biret Date: Thu, 5 Sep 2024 14:22:11 -0400 Subject: [PATCH] ci: removes outdated branch policy for dev branch Signed-off-by: Vincent Biret --- ...raph-sdk-python-core-branch-protection.yml | 37 ------------------- .github/workflows/build.yml | 2 +- .github/workflows/codeql-analysis.yml | 6 +-- .github/workflows/conflicting-pr-label.yml | 4 +- 4 files changed, 6 insertions(+), 43 deletions(-) diff --git a/.github/policies/msgraph-sdk-python-core-branch-protection.yml b/.github/policies/msgraph-sdk-python-core-branch-protection.yml index 43f80a42..39f32475 100644 --- a/.github/policies/msgraph-sdk-python-core-branch-protection.yml +++ b/.github/policies/msgraph-sdk-python-core-branch-protection.yml @@ -7,43 +7,6 @@ resource: repository configuration: branchProtectionRules: - - branchNamePattern: dev - # This branch pattern applies to the following branches as of 06/12/2023 11:37:28: - # dev - - # Specifies whether this branch can be deleted. boolean - allowsDeletions: false - # Specifies whether forced pushes are allowed on this branch. boolean - allowsForcePushes: false - # Specifies whether new commits pushed to the matching branches dismiss pull request review approvals. boolean - dismissStaleReviews: true - # Specifies whether admins can overwrite branch protection. boolean - isAdminEnforced: false - # Indicates whether "Require a pull request before merging" is enabled. boolean - requiresPullRequestBeforeMerging: true - # Specifies the number of pull request reviews before merging. int (0-6). Should be null/empty if PRs are not required - requiredApprovingReviewsCount: 1 - # Require review from Code Owners. Requires requiredApprovingReviewsCount. boolean - requireCodeOwnersReview: true - # Are commits required to be signed. boolean. TODO: all contributors must have commit signing on local machines. - requiresCommitSignatures: false - # Are conversations required to be resolved before merging? boolean - requiresConversationResolution: true - # Are merge commits prohibited from being pushed to this branch. boolean - requiresLinearHistory: false - # Required status checks to pass before merging. Values can be any string, but if the value does not correspond to any - # existing status check, the status check will be stuck on pending for status since nothing exists to push an actual status - requiredStatusChecks: - - CodeQL - - check-build-matrix - - license/cla - # Require branches to be up to date before merging. Requires requiredStatusChecks. boolean - requiresStrictStatusChecks: true - # Indicates whether there are restrictions on who can push. boolean. Should be set with whoCanPush. - restrictsPushes: false - # Restrict who can dismiss pull request reviews. boolean - restrictsReviewDismissals: false - - branchNamePattern: main # This branch pattern applies to the following branches as of 01/12/2024 13:14:28: # main diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 31b659d9..8cdaf75d 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -7,7 +7,7 @@ on: push: branches: [main] pull_request: - branches: [main, dev] + branches: [main] workflow_call: jobs: diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml index 44ed1c0f..12abf02a 100644 --- a/.github/workflows/codeql-analysis.yml +++ b/.github/workflows/codeql-analysis.yml @@ -13,10 +13,10 @@ name: "CodeQL" on: push: - branches: [main, dev] + branches: [main] pull_request: # The branches below must be a subset of the branches above - branches: [main, dev] + branches: [main] schedule: - cron: "32 11 * * 6" @@ -66,4 +66,4 @@ jobs: # ./location_of_script_within_repo/buildscript.sh - name: Perform CodeQL Analysis - uses: github/codeql-action/analyze@v3 \ No newline at end of file + uses: github/codeql-action/analyze@v3 diff --git a/.github/workflows/conflicting-pr-label.yml b/.github/workflows/conflicting-pr-label.yml index 998fb559..86ba0302 100644 --- a/.github/workflows/conflicting-pr-label.yml +++ b/.github/workflows/conflicting-pr-label.yml @@ -6,10 +6,10 @@ name: PullRequestConflicting # events but only for the master branch on: push: - branches: [main, dev] + branches: [main] pull_request: types: [synchronize] - branches: [main, dev] + branches: [main] permissions: pull-requests: write