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

Reply via email to