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

[RM chapter 5] Language corrections. #10

Merged

Conversation

mpgreaves
Copy link
Contributor

No description provided.

@walterkozlowski walterkozlowski merged commit 1659509 into anuket-project:main Feb 6, 2024
1 check passed
@walterkozlowski walterkozlowski requested review from a team and bfcohen February 28, 2024 05:45
@walterkozlowski
Copy link
Collaborator

@bfcohen, can you please review these language corrections, which I approved? You are our English and editing expert in residence!

@bfcohen
Copy link
Contributor

bfcohen commented Feb 28, 2024

Have to log in to the correct account.

Copy link

@bfcohen100 bfcohen100 left a comment

Choose a reason for hiding this comment

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

Needs some more editing. I will complete the review and then reopen to do the edits. I think that will be easier.

Choose a reason for hiding this comment

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

Needs some more edits. I have to reopen the file to do the edits.


Cloud Infrastructure Software profile description
Cloud Infrastructure node (host or server). :ref:`chapters/chapter02:profile extensions (specialisations)`
may specify additional configurations. Workloads utilise profiles to describe the configuration of nodes on which they

Choose a reason for hiding this comment

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

replace utilise with use

Copy link
Collaborator

Choose a reason for hiding this comment

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

fixed in #17

Copy link

@bfcohen100 bfcohen100 left a comment

Choose a reason for hiding this comment

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

Needs more edits

Copy link
Contributor

@bfcohen bfcohen left a comment

Choose a reason for hiding this comment

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

Overall looks OK. Needs some polishing.

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.

5 participants