⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Add a step for syncing README.md to Docker Hub in container-image.yml #8

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
yhabteab wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add a step for syncing README.md to Docker Hub in container-image.yml #8

yhabteab wants to merge 1 commit into from

Conversation

@yhabteab
Copy link
Member

@yhabteab yhabteab commented Aug 5, 2025 •
edited
Loading

@yhabteab yhabteab requested a review from julianbrost August 5, 2025 12:10
@cla-bot cla-bot bot added the cla/signed CLA is signed by all contributors of a PR label Aug 5, 2025
@yhabteab yhabteab force-pushed the sync-image-readme-md branch 3 times, most recently from 6454960 to 907df68 Compare August 5, 2025 13:07
@yhabteab yhabteab changed the title Add a reusable GHA for syncing README.md to Docker Hub Add a step for syncing README.md to Docker Hub in container-image.yml Aug 5, 2025
@yhabteab yhabteab force-pushed the sync-image-readme-md branch from 907df68 to c3615ec Compare August 5, 2025 13:24
@yhabteab yhabteab mentioned this pull request Aug 8, 2025
Merged
julianbrost
julianbrost reviewed Aug 8, 2025
View reviewed changes
@julianbrost
Copy link

julianbrost commented Aug 8, 2025

Other idea (originating from Icinga/icinga2#10505 (comment)): The documentation mostly describes the latest tag, like the examples refer to something like icinga/icinga2 (implicitly :latest), so what about syncing when uploading that tag (i.e. on: release) instead of for pushes to the default branch?

@yhabteab yhabteab force-pushed the sync-image-readme-md branch from c3615ec to cadb12d Compare August 8, 2025 11:52
@yhabteab yhabteab requested a review from julianbrost August 8, 2025 11:53
julianbrost
julianbrost reviewed Aug 29, 2025
View reviewed changes
@yhabteab yhabteab force-pushed the sync-image-readme-md branch from cadb12d to 6f9b5e4 Compare August 29, 2025 16:21
@julianbrost julianbrost self-assigned this Sep 16, 2025
@julianbrost
Copy link

julianbrost commented Oct 16, 2025

Did you test this again after the last round of changes? Then it should be fine.

@yhabteab yhabteab removed their assignment Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@julianbrost julianbrost julianbrost left review comments

Assignees

@julianbrost julianbrost

Labels

cla/signed CLA is signed by all contributors of a PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yhabteab @julianbrost