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

[Tooling] Protect the release/* branch with the same settings as main #3562

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

AliSoftware
Copy link
Contributor

During code freeze and after the creation of the new release/* branch, replaces the call to set_branch_protection with copy_branch_protection instead, so that it replicates the same branch protections as the ones set on the main branch.

In particular, this will require the same CI checks on PRs targeting the release/* branch than the ones set for the main branch

And in particular enabling the required CI checks on PRs targeting the release branch too
@AliSoftware AliSoftware added [Type] Tooling Related to the Gradle build scripts and the setup or maintenance of the project build process. [Area] Tooling labels Feb 6, 2025
@AliSoftware AliSoftware added this to the 7.83 milestone Feb 6, 2025
@AliSoftware AliSoftware requested a review from a team February 6, 2025 18:27
@AliSoftware AliSoftware self-assigned this Feb 6, 2025
@AliSoftware AliSoftware requested a review from a team as a code owner February 6, 2025 18:27
@AliSoftware AliSoftware requested review from MiSikora and removed request for a team February 6, 2025 18:27
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Feb 6, 2025

📲 You can test the changes from this Pull Request in 📱 Mobile by scanning the QR code below to install the corresponding build.
App Name 📱 Mobile
Build TypedebugProd
Commite8caec8
Direct Downloadpocketcasts-app-prototype-build-pr3562-e8caec8.apk
📲 You can test the changes from this Pull Request in 🚗 Automotive by scanning the QR code below to install the corresponding build.
App Name 🚗 Automotive
Build TypedebugProd
Commite8caec8
Direct Downloadpocketcasts-automotive-prototype-build-pr3562-e8caec8.apk
📲 You can test the changes from this Pull Request in ⌚ Wear by scanning the QR code below to install the corresponding build.
App Name ⌚ Wear
Build TypedebugProd
Commite8caec8
Direct Downloadpocketcasts-wear-prototype-build-pr3562-e8caec8.apk

@AliSoftware AliSoftware merged commit 3be67d7 into main Feb 6, 2025
20 checks passed
@AliSoftware AliSoftware deleted the tooling/release-branch-protection branch February 6, 2025 18:50
# restrictions we have in `main`, and the release managers wouldn't be able to push due to permissions.
# This should be changed only when we have PC Android releases done on CI, when the CI bot is the one running `git push`.
set_branch_protection(repository: GITHUB_REPO, branch: "release/#{new_version}")
copy_branch_protection(repository: GITHUB_REPO, branch: "release/#{new_version}", from_branch: DEFAULT_BRANCH)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shoot, that's not the name of the parameters expected by that action… I shouldn't have trusted Cursor's autocomplete suggestion 😓

Fix in #3563

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Area] Tooling [Type] Tooling Related to the Gradle build scripts and the setup or maintenance of the project build process.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants