Skip to content

Make CI checks to always merge into current main/master - #789

Open
alfsb wants to merge 64 commits into
php:masterfrom
alfsb:action-checkout-always-merge-master-head
Open

Make CI checks to always merge into current main/master#789
alfsb wants to merge 64 commits into
php:masterfrom
alfsb:action-checkout-always-merge-master-head

Conversation

@alfsb

@alfsb alfsb commented Jul 29, 2026

Copy link
Copy Markdown
Member

This PR recreates CI tests, using plain git, instead of Github actions/checkout, that ignores changes on the main/master in checkout step, only in the checkout of repository related to the PR. Because of that, CI results are almost always outdated or invalid in busy repositories.

@alfsb

alfsb commented Aug 8, 2026

Copy link
Copy Markdown
Member Author

Novas alterações enviadas no main/master, é possível re-testar a implementação. Antes de re-rodar os workflows, os hashes computados para os manuais gerados eram:

83feb956db564741733c1b69cec02cc59a994d5e  doc-base/temp/manual.xml
83feb956db564741733c1b69cec02cc59a994d5e  doc-base/temp/manual.xml

e depois

0e9bc8f1d383c1732dd5d34f54f635a8371b09a1  doc-base/temp/manual.xml
b860020de2783afce3c728eed35409f7441969d5  doc-base/temp/manual.xml

Onde é possível observar que, sim, a nova implementação não só conseguiu fazer o merge corretamente, como gerou um estado diferente dos arquivos. A seguir, vou encaminhar mais uma alteração, que tenta minimizar a implementação, que está muito longa.

@alfsb alfsb changed the title GH actions/checkout dont always merge into current main/master Make CI checks to always merge into current main/master Aug 8, 2026
@alfsb

alfsb commented Aug 8, 2026

Copy link
Copy Markdown
Member Author

Novamente, desculpe pelo ruído. Refiz a versão em um arquivo separado, para facilitar comparação e implantação. Eu planejo esperar um pouco, até que o ramo padrão receba atualizações, e confirmar que a implementação alternativa gera resultados idênticos. Daí um último push remove o arquivo antigo.

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