Skip to content

Conversation

@m-blaha
Copy link
Member

@m-blaha m-blaha commented Feb 6, 2026

@centosinfra-prod-github-app
Copy link
Contributor

Copy link
Member

@nforro nforro left a comment

Choose a reason for hiding this comment

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

Could you please also add a config example?

@m-blaha
Copy link
Member Author

m-blaha commented Feb 9, 2026

I've added examples to both bodhi_update.md and examples.md. Is it too much?

@centosinfra-prod-github-app
Copy link
Contributor

@nforro
Copy link
Member

nforro commented Feb 9, 2026

I've added examples to both bodhi_update.md and examples.md. Is it too much?

I don't think so, but I would limit the first example to the option itself and move it to its description. Or maybe we can just say Refer to the configuration examples for more information., like in the desctiption of tf_extra_params.

@m-blaha
Copy link
Member Author

m-blaha commented Feb 9, 2026

Agreed. Simple reference would be enough.

Signed-off-by: Marek Blaha <mblaha@redhat.com>
Signed-off-by: Marek Blaha <mblaha@redhat.com>
@centosinfra-prod-github-app
Copy link
Contributor

@m-blaha m-blaha added the mergeit Merge via Zuul label Feb 9, 2026
@centosinfra-prod-github-app
Copy link
Contributor

@centosinfra-prod-github-app centosinfra-prod-github-app bot merged commit 63a3058 into packit:main Feb 9, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Packit pull requests Feb 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mergeit Merge via Zuul

Projects

Development

Successfully merging this pull request may close these issues.

2 participants