search

stellar / system-test

Home of the stellar/stellar-system-test docker image for e2e system testing
1 stars 5 forks source link

fixed gha workflow to push to dockerhub on pr merge #9

Closed sreuland closed 1 year ago

sreuland commented 1 year ago

need the pr merge event to trigger the dockerhub publish step

sreuland commented 1 year ago

LGTM, but isn’t this already implied by the “on: push: master” at the top of the Action?

this filters out some PR events, otherwise pr's from remote repos, were falling into dockerhub push also, and intent was to only allow pr's from same repo(not forks) or master pushes to result in push to docker hub also