Skip to content

Conversation

@ibarrajo
Copy link
Contributor

This updates the Github action to create a PR for data updates instead of creating commits directly to master branch.

We can't commit directly to master branch due to branch protection rules.

The upside is that all data updates will be reviewed before deployment.

Example of PR:
image

PR and branch can be deleted and the github action will recreate it if needed using the default branch as the base, this is useful in case history diverges or there is stale data on the branch.

@ibarrajo ibarrajo requested a review from a team March 12, 2025 20:27
ref: ${{ env.BRANCH }}

- name: Fallback to default branch if checkout failed
if: steps.try-checkout.outcome == 'failure'
Copy link
Member

Choose a reason for hiding this comment

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

The potential problem with this approach is that once you create fetch-release-data-update branch and forget to delete it, this workflow will always use that instead of default branch. Might be fine but please document this behavior somewhere.

@ibarrajo ibarrajo merged commit 3f41fff into cadence-workflow:master Mar 20, 2025
1 check passed
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.

2 participants