search

pressbooks / composer-autoupdate-bedrock

Auto updater action 🤖
0 stars 0 forks source link

Fix autoupdate and automerge to work with new branch/PR linting rules #64

Closed SteelWagstaff closed 1 year ago

SteelWagstaff commented 1 year ago

The commits and PRs automatically created by our composer-autoupdate action no longer pass our globally required branch and PR linting rules. See https://github.com/pressbooks/pressbookspublic-bedrock/pull/569 for a failing PR.

We should update our composer-autoupdate-bedrock action so that the PR branch name and commits match our new rules (i.e. the branch is called something like chore/dev-updated and the commit and PR titles look like chore: composer update with 4 changes. See the GIT_COMMIT_PREFIX command available here: https://github.com/kawax/composer-update-action#env. i don't know how to make the branch name match the regex -- i only see a suffix command in the parent action.

We'll also need to adjust https://github.com/pressbooks/composer-autoupdate-bedrock/blob/028a75871a2de0ccd5e67e2e8ef7de3bf31438f0/.github/workflows/auto-merge.yml#L7

arzola commented 1 year ago

WIP https://github.com/pressbooks/composer-autoupdate-bedrock/blob/testingv3/.github/workflows/auto-merge.yml

arzola commented 1 year ago

https://github.com/pressbooks/composer-autoupdate-bedrock/commit/717b7d3d6c72dddf86b7c7b8ae0c71890f80d16c

This should be transparent for our repos, I updated the previous tag (v2) to avoid changing the pipelines in our repos

Composer autoupdate PRs would have the title: "chore: Composer update with N changes"