-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add create setting file step to Katello branch procedure #270
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.
Regarding the place: I think #214 would use it to build the tool_belt config, so adding it prior to that would be better.
procedures/katello/branch.md.erb
Outdated
@@ -49,6 +49,7 @@ | |||
- [ ] [pulpcore](https://github.com/theforeman/puppet-pulpcore) | |||
- [ ] [foreman_proxy_content](https://github.com/theforeman/puppet-foreman_proxy_content) | |||
- [ ] [katello](https://github.com/theforeman/puppet-katello) | |||
- [ ] Commit the [settings file](https://github.com/theforeman/theforeman-rel-eng/blob/master/releases/katello/<%= release %>/settings) to the `theforeman-rel-eng` repository |
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.
Would it make sense to add a note about how to create it? or link to a commit that adds it for a previous version?
8081d8e
to
434a83f
Compare
@ekohl updated, let me know how this looks please. |
procedures/katello/branch.md.erb
Outdated
@@ -33,6 +33,8 @@ | |||
|
|||
## Release Owner | |||
|
|||
- [ ] Commit the [settings file](https://github.com/theforeman/theforeman-rel-eng/blob/master/releases/katello/<%= release %>/settings) to the `theforeman-rel-eng` repository | |||
- [ ] See an example [here](https://github.com/theforeman/theforeman-rel-eng/commit/29b6b92e82de25c7a0e32fb9ca36468d8df0f6a1) |
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'd not add a checkbox here since there's no real action to be taken. Perhaps even put it in a single line?
434a83f
to
8080b38
Compare
@ekohl updated to put it on one line, like you suggested. |
@ekohl
Let me know if this is the correct place to put this