-
Notifications
You must be signed in to change notification settings - Fork 11
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
debatable improvements #3
debatable improvements #3
Conversation
3ab8f0e
to
a5bd62a
Compare
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.
Thanks for the contribution. Lots of nice fix ups.
However, I don't think we would capitalize "bootable contaner images" and "image based linux" where ever they appear in the text. Agree we can capitalize in titles though.
Could you also add a Signed-off-by
? I'm adding a description of this in #4
a5bd62a
to
9ccf1e3
Compare
Signed-off-by: Simon de Vlieger <[email protected]>
The sentence has a 'both' in there. Signed-off-by: Simon de Vlieger <[email protected]>
Signed-off-by: Simon de Vlieger <[email protected]>
9ccf1e3
to
4807343
Compare
Signed-off-by: Simon de Vlieger <[email protected]>
4807343
to
ff82655
Compare
I've backed out the capitalization commit and replaced it with one that only capitalizes in the title :) |
Doing these as separate commits as each can probably be discussed/removed from the PR separately. Most of these are likely opinionated as well.