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

Fixes #37654 - Use custom snippets to apply errata #3164

Merged
merged 2 commits into from
Jul 29, 2024

Conversation

maximiliankolb
Copy link
Contributor

Refs 2064ec7457b3d41b72c2d48b7df9371f8b09d053 in katello

Closes #3157

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Do not cherry-pick.

@pr-processor pr-processor bot added Waiting on contributor Requires an action from the author and removed Not yet reviewed labels Jul 25, 2024
@pr-processor pr-processor bot added Needs re-review and removed Waiting on contributor Requires an action from the author labels Jul 26, 2024
Copy link
Contributor Author

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

Rebased to "master" and applied all feedback. I also applied some feedback to the file where I "borrowed" the skeleton from. Ready for re-review @apinnick

Do you have a suggestion for a better title? The snippets should not be used to apply errata, but to customize applying errata on hosts.

@pr-processor pr-processor bot added Waiting on contributor Requires an action from the author and removed Needs re-review labels Jul 28, 2024
@pr-processor pr-processor bot added Needs re-review and removed Waiting on contributor Requires an action from the author labels Jul 29, 2024
Copy link
Contributor Author

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

Applied feedback; ready for re-review.

Copy link
Contributor

@apinnick apinnick left a comment

Choose a reason for hiding this comment

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

LGTM

@maximiliankolb maximiliankolb merged commit 852a5f2 into theforeman:master Jul 29, 2024
9 checks passed
@maximiliankolb maximiliankolb deleted the fix_3157_custom_snippets branch July 29, 2024 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create docs around Use custom pre/post snippets for errata templates
2 participants