Skip to content

Conversation

@metachris
Copy link
Contributor

@metachris metachris commented Feb 10, 2026

Build Docker image, and push to Github container registry

@metachris metachris requested review from ameba23 and Copilot February 10, 2026 17:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new GitHub Actions job to build and push a Docker image to GitHub Container Registry when a release is created. The job is triggered on push events or when a manual workflow dispatch includes the build-docker input set to true.

Changes:

  • Added a new build-docker job that builds and pushes a multi-platform Docker image
  • Configured Docker metadata extraction to generate appropriate semantic version tags
  • Set up GitHub Container Registry authentication using the GitHub token

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@ameba23 ameba23 left a comment

Choose a reason for hiding this comment

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

Great to have this in! 💯

I made a test release on this branch and can confirm i was able to docker pull ghcr.io/flashbots/attested-tls-proxy:baca665

Feel free to delete the test release and associated container.

Probably it is worth addressing the issues copilot flagged as i checked it over with chatgpt and it also mentioned something about releases named directly as hashes without a sha- prefix may be unintentionally marked as "latest". 🤷

@metachris
Copy link
Contributor Author

Amazing! Thanks @ameba23. I've applied the suggested changes, think they were good suggestions 👍

@metachris metachris merged commit bb27816 into main Feb 11, 2026
2 checks passed
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