vy commented on PR #1322:
URL: https://github.com/apache/logging-log4j2/pull/1322#issuecomment-1454866093

   Great job @ppkarwasz! :100: AFAIK, this setup would run the build workflow 
twice: the 1st one while verifying the PR, and the 2nd one after merging it. 
You can avoid this by adding the following block to `build.yml`:
   
   ```
   on:
     push:
       # Avoid workflow run for _merged_ `dependabot` PRs.
       # They were (hopefully!) already tested in PR-triggered workflow.
       branches-ignore: "dependabot/**"
   ```
   
   See [my rotating-fos 
configuration](/rotating-fos/blob/master/.github/workflows/build.yml#L21) for 
an example.
   
   You can this PR after this change, IMO.


-- 
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