Skip to content
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.

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

Fixing broken link for container-registries.conf #1153

Closed
wants to merge 1 commit into from

Conversation

rhatdan
Copy link
Member

@rhatdan rhatdan commented Nov 1, 2018

The link to the container-registries.conf man page was a little off - this fixes it.

This patch was originaly submitted by Kris Nova kris@nivenly.com
I am signing it to get it merged.

Signed-off-by: Daniel J Walsh dwalsh@redhat.com

The link to the container-registries.conf man page was a little off - this fixes it.

This patch was originaly submitted by Kris Nova <kris@nivenly.com>
I am signing it to get it merged.

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
@rhatdan
Copy link
Member Author

rhatdan commented Nov 1, 2018

Replaces #1116
Thank you @kris-nova

Copy link
Member

@TomSweeneyRedHat TomSweeneyRedHat left a comment

Choose a reason for hiding this comment

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

LGTM

@rhatdan
Copy link
Member Author

rhatdan commented Nov 2, 2018

@rh-atomic-bot r+

@rh-atomic-bot
Copy link
Collaborator

📌 Commit f934b4a has been approved by rhatdan

@TomSweeneyRedHat
Copy link
Member

@rhatdan greet test buttons, but it doesn't seem to have merged auto-magically as I'd expect?

@rhatdan
Copy link
Member Author

rhatdan commented Nov 2, 2018

I think humu is waiting on a different PR to merge.

@rh-atomic-bot
Copy link
Collaborator

⚡ Test exempted: pull fully rebased and already tested.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants