-
Notifications
You must be signed in to change notification settings - Fork 120
Trivial editing #305
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
Trivial editing #305
Conversation
| CentOS 7 on your hardware. The defaults will generally be acceptable for this | ||
| installation - but make sure to configure IP address/parameters so that you can later install needed | ||
| packages from internet. Later, we will change the network configuration as needed. | ||
| packages from the internet. Later, we will change the network configuration as needed. |
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 am not an English speaker but isn't it either internet or the Internet ? (note the capitalisation)
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'm an English speaker, and it would be "the internet" or "the Internet". English isn't a single language though.
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.
@bradh, so capitalisation is optional if the article is used?
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.
Nouns (other than pronouns) are not usually capitalised.
Seriously though, its not a big deal. The only thing I was trying to do in this PR was fix the spelling mistake. I can back out the rest 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.
no, don´t bother I'll merge
| $ alternatives --config java | ||
| Make sure that Java 11 is the chosen one. | ||
| Make sure that Java 11 is selected. |
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 kind of like the chosen one ;)
* Update service offering section with new compute offering and disk offering changes (#238) * Added sample ssh commands to connect to Control and Worker node (#304) Co-authored-by: dahn <daan.hoogland@gmail.com> Co-authored-by: Abhishek Kumar <abhishek.mrt22@gmail.com> * Trivial editing (#305) * volume migration docs for PowerFlex/ScaleIO storage pool * Rename VM to Instance * Update storage.rst package update limitation * revert changes, as it's not needed storage.rst --------- Co-authored-by: kiranchavala <kiranchavala@gmail.com> Co-authored-by: dahn <daan.hoogland@gmail.com> Co-authored-by: Abhishek Kumar <abhishek.mrt22@gmail.com> Co-authored-by: Brad Hards <bradh@frogmouth.net> Co-authored-by: Boris Stoyanov - a.k.a Bobby <bss.stoyanov@gmail.com>
No description provided.