Skip to content

Housekeeping can't work when run on merge in svalinn (no PR number)#757

Merged
gonuke merged 2 commits into
svalinn:developfrom
bam241:fix_housekeeping_test
Jul 18, 2021
Merged

Housekeeping can't work when run on merge in svalinn (no PR number)#757
gonuke merged 2 commits into
svalinn:developfrom
bam241:fix_housekeeping_test

Conversation

@bam241

@bam241 bam241 commented Jul 18, 2021

Copy link
Copy Markdown
Member

this prevent housekeeping to run on merge for svalinn org

@gonuke gonuke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick fix @bam241

@gonuke gonuke merged commit 9d989e0 into svalinn:develop Jul 18, 2021
tags: ghcr.io/${{ github.repository_owner }}/dagmc-ci-ubuntu-${{ matrix.ubuntu_versions }}-${{ matrix.compiler}}-ext-hdf5_${{ matrix.hdf5_versions}}-moab_${{ matrix.moab_versions }}

housekeeping:
if: ${{ github.repository_owner != 'svalinn' || github.event_name == pull_request }}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was too quick!!! this needs quotes 'pull_request' - sorry!

This was referenced Jul 19, 2021
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