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
Fixes #19663 - format foreman installer options as a list in the docs #456
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to do the same thing in more places but this is a good one to start.
docs/development.md
Outdated
@@ -34,7 +34,11 @@ centos7-devel: | |||
group: 'devel' | |||
variables: | |||
katello_devel_github_username: <REPLACE ME> | |||
foreman_installer_options_internal_use_only: "--katello-devel-github-username myname --katello-devel-upstream-remote-name origin --katello-devel-extra-plugins theforeman/foreman_remote_execution --katello-devel-extra-plugins theforeman/foreman_discovery" | |||
foreman_installer_options_internal_use_only: | |||
- "--katello-devel-github-username <REPLACE ME>" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if you can use {{ katello_devel_github_username }}
at this point.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably we can, will test in a second. Thanks for the suggestion!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Given we override the foreman_installer_options_internal_use_only
variable it looks like katello_devel_github_username
isn't used at all actually. This example can be simplified to get rid of it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought about that too. I prefer to use {{ katello_devel_github_username }}
as that allows us to yank the foreman_installer_options_internal_use_only
at a later point and still have the username set properly.
Thanks! |
No description provided.