How to make a combined action run on either PR or Push event? #26786

iamfotx · 2020-08-09T15:09:07Z

iamfotx
Aug 9, 2020

I need a workflow, where I want action to run on each PR against master. And I also want to run that same action when a push (direct commit not PR merge commit) happens in master.

Current configurations looks like this:

The Problem:
It makes those actions run twice i.e

When PR is being created against master (that makes sense as I’m listening to on PR event)
When that PR gets merged into master (I don’t want to re run the action this time as It’s already in passing state from PR)

Expected Behavior:
I’m expecting action to not run twice, I want to run the action like this:

When a PR gets opened against master, run the action.
When a direct commit happens to master (again not on the PR merge commit as it will make the action to run twice).

Answered by BrightRan

Aug 10, 2020

@iamfotx,

When a commit is pushed from a merged PR, the automatically generated the head commit message generally follows the format “Merge pull request #<pr_number> from /<source_branch>\n\n<pr_title>”.
For example:

head_commit_message998×119 10.6 KB

You can use the expression “github.event.head_commit.message” to get the head commit message from the github context.
You can use the following if conditional to skip all the jobs in the workflow when the push is from the PR merged.

if: ${{ github.event_name == 'pull_request' || (github.event_name == 'push' && !contains( github.event.head_commit.message, 'Merge pull request')) }}

A simple demo:

on:
  push:
    branches:
      - master
  p…

View full answer

BrightRan · 2020-08-10T07:25:19Z

BrightRan
Aug 10, 2020

@iamfotx,

When a commit is pushed from a merged PR, the automatically generated the head commit message generally follows the format “Merge pull request #<pr_number> from /<source_branch>\n\n<pr_title>”.
For example:

head_commit_message998×119 10.6 KB

You can use the expression “github.event.head_commit.message” to get the head commit message from the github context.
You can use the following if conditional to skip all the jobs in the workflow when the push is from the PR merged.

if: ${{ github.event_name == 'pull_request' || (github.event_name == 'push' && !contains( github.event.head_commit.message, 'Merge pull request')) }}

A simple demo:

on:
  push:
    branches:
      - master
  pull_request:
    branches:
      - master
jobs:

job1:

if: ${{ github.event_name == 'pull_request' || (github.event_name == 'push' && !contains( github.event.head_commit.message, 'Merge pull request')) }}

. . .
job2:

if: ${{ github.event_name == 'pull_request' || (github.event_name == 'push' && !contains( github.event.head_commit.message, 'Merge pull request')) }}

. . .
jobN:

if: ${{ github.event_name == 'pull_request' || (github.event_name == 'push' && !contains( github.event.head_commit.message, 'Merge pull request')) }}

. . .

0 replies

iamfotx · 2020-08-10T18:48:55Z

iamfotx
Aug 10, 2020
Author

Thank you so much for the reply! It worked like charm.

But there’s a problem with git checks:

This is how my commit looks in an Active PR…

image856×27 3.87 KB

When I merge this PR into master:

I’ve lost this green status check mark. And I think when using circleci/codeship, we didn’t to re-run the workflow on the master branch again rather it picks the status of the PR.

0 replies

airtower-luna · 2020-08-10T19:04:39Z

airtower-luna
Aug 10, 2020

The second screenshot doesn’t have a commit ID, but I assume it’s showing the merge commit? If so, that’s exactly why there is no check mark: The workflow didn’t run for the merge commit. When you look at the commit history the original commit should still have the check mark.