search

matrix-org / matrix-appservice-irc

Node.js IRC bridge for Matrix
Apache License 2.0
460 stars 149 forks source link

docker: don't push on a Pull Request #1767

Closed gdamjan closed 11 months ago

gdamjan commented 11 months ago

Since the github action triggers on a PR too, github.ref will be the target branch for merging (typically develop). In that case, don't try to login/push to docker hub.

https://docs.github.com/en/actions/learn-github-actions/contexts#github-context

Signed-off-by: Damjan Georgievski gdamjan@gmail.com

Half-Shot commented 11 months ago

Looks good, happy to merge.

Half-Shot commented 11 months ago

Thanks again :)