ppkarwasz commented on PR #379: URL: https://github.com/apache/logging-parent/pull/379#issuecomment-2804521321
Since changes to reusable workflows are half of the PRs in this repository, I find that this is not just a convenience for maintainers: - A PR can be approved by mistake, because the checks didn't run the reusable workflows modified by the PR. - If the author modifies `build.yaml` to run the reusable workflow modified by the PR, he might forget changing it back before the merge. This will result in a broken `build.yaml` as soon as the PR branch is deleted. I could improve the workflow documentation in the website. What do you think? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@logging.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org