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

[bitnami/rabbitmq] Update README.md #32827

Closed
wants to merge 1 commit into from

Conversation

thecatontheflat
Copy link

Fixes outdated link

Signed-off-by: Vitalii Zurian <thecatontheflat@users.noreply.github.com>
@github-actions github-actions bot added rabbitmq triage Triage is needed labels Apr 6, 2025
@github-actions github-actions bot requested a review from carrodher April 6, 2025 07:39
@javsalgar javsalgar changed the title Update README.md [bitnami/rabbitmq] Update README.md Apr 7, 2025
@javsalgar javsalgar added verify Execute verification workflow for these changes in-progress labels Apr 7, 2025
@github-actions github-actions bot removed the triage Triage is needed label Apr 7, 2025
@github-actions github-actions bot removed the request for review from carrodher April 7, 2025 11:54
@github-actions github-actions bot requested a review from migruiz4 April 7, 2025 11:54
@migruiz4
Copy link
Member

migruiz4 commented Apr 8, 2025

Hi @thecatontheflat, thank you for reporting this issue!

In this case, the link in the bitnami/rabbitmq README is correct, but I think we accidentally removed the parameters section in the bitnami/common README.

I think the correct fix here would be to restore the common README instead of patching the affected links.

@migruiz4
Copy link
Member

migruiz4 commented Apr 8, 2025

I have created this PR restoring the missing section: #32861

@thecatontheflat
Copy link
Author

Thanks! I am closing this PR then

@thecatontheflat thecatontheflat deleted the patch-1 branch April 8, 2025 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rabbitmq solved verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants