Skip to content

Conversation

mkoeppe
Copy link
Contributor

@mkoeppe mkoeppe commented May 11, 2024

Sometimes I push mock release tags to my repo to test workflows relevant to making a release.

Currently, the workflow "Build Docker images and push to DockerHub" fails in this situation when it tries to log in to DockerHub; example: https://github.com/mkoeppe/sage/actions/runs/9045501088/job/24855325483

This PR modifies it to just skip the "login" step when the secrets are not set. It then still builds the image, but does not try to push. Example: https://github.com/mkoeppe/sage/actions/runs/9046681672/job/24857769764

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation and checked the documentation preview.

⌛ Dependencies

Copy link

github-actions bot commented May 11, 2024

Documentation preview for this PR (built with commit ffabaaf; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

@soehms
Copy link
Member

soehms commented May 13, 2024

I will have a look at it in connection with #37856 (comment).

Copy link
Member

@soehms soehms left a comment

Choose a reason for hiding this comment

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

Thanks!

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 22, 2024

Thank you!

@vbraun vbraun merged commit f668757 into sagemath:develop May 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants