Skip to content

Comments

Fixed typos and grammar mistakes (3.27)#3599

Open
olehermanse wants to merge 1 commit intocfengine:3.27from
olehermanse:3.27
Open

Fixed typos and grammar mistakes (3.27)#3599
olehermanse wants to merge 1 commit intocfengine:3.27from
olehermanse:3.27

Conversation

@olehermanse
Copy link
Member

(cherry picked from commit 5f9112d)

Co-authored-by: Claude <noreply@anthropic.com>
(cherry picked from commit 5f9112d)
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
Comment on lines +71 to +74
By default, `cf-execd` emails a summary of any `cf-agent` run. You
may want to adjust the mailto or mailfrom. If you have a centralized reporting
system like CFEngine Enterprise you may wish to disable agent emails all
together.
system like CFEngine Enterprise you may wish to disable agent emails
altogether.
Copy link
Contributor

Choose a reason for hiding this comment

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

If I was editing this I would also collapse sentences to one-per-line. That would obscure the typo fix though so maybe two commits. :p

1500 IOPS and 10.5 MB/s).

**Note** Your storage IOPS specification may be given in 4KiB block
**Note:** Your storage IOPS specification may be given in 4KiB block
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this done other places? Note:

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, there are many instances of just Note: and then also of **Note:** so maybe we should make them all the same? One way or another I agree with a semi-colon following but I am not sure about whether to add the asterisks or not.


Step 1. consists of the following sequence: First, launch PuTTYgen by double-clicking on the puTTygen icon in the Windows programs menu tree;
(It should be inside the PuTTY folder that was created when the PuTTY was downloaded and installed.)
Step 1 consists of the following sequence: First, launch PuTTYgen by double-clicking on the PuTTYgen icon in the Windows programs menu tree;
Copy link
Contributor

Choose a reason for hiding this comment

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

This is step 1 stuff, should be moved up to step 1!?

Copy link
Contributor

@craigcomstock craigcomstock left a comment

Choose a reason for hiding this comment

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

oops. I just noticed this is a cherry-pick. I made a few comments. So those could be addressed in master and then cherry-picked. As you wish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants