-
Notifications
You must be signed in to change notification settings - Fork 247
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
config/doc/ignition.yaml: correction in description #1930
Conversation
570383f
to
0d91393
Compare
docs/release-notes.md
Outdated
@@ -16,6 +16,7 @@ nav_order: 9 | |||
### Changes | |||
|
|||
- The Dracut module now installs partx | |||
- Sentence correction in document `luks.cex.enabled.desc` |
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 can drop this, I dont think we need to add release notes for this change.
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.
Done.
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.
After you drop the release-notes.md change lgtm, thank you for doing this.
Docs rephrase in luks.cex.enabled description. coreos/butane#536 (comment)
0d91393
to
d11012b
Compare
Hi @prestist , Without the release notes it fails the test. Is there any way to skip this? |
We could just ignore the test output and merge? |
@madhu-pillai there is a label for this, I just added it thank you! |
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.
LGTM
This PR is raised to correct the sentence in
desc:
for CEX feature in ignition as well as butane.Ref: coreos/butane#536 (comment)