Fix typographical errors in services section #198

Merged
pboy merged 1 commit from brettweir/user-documentation:proofread-services into main 2026-07-15 20:53:23 +00:00
Member

This pull request makes the following changes to the services section of the documentation:

  • Fix spelling and punctuation errors.
  • Switch to US English spellings where applicable.
  • Avoid the use of hidden/ambiguous Unicode characters where ASCII characters would suffice.

I tried to take a light-touch approach here, but happy to update the PR if any changes are too aggressive.

This pull request makes the following changes to the **services** section of the documentation: - Fix spelling and punctuation errors. - Switch to US English spellings where applicable. - Avoid the use of hidden/ambiguous Unicode characters where ASCII characters would suffice. I tried to take a light-touch approach here, but happy to update the PR if any changes are too aggressive.
Assisted-by: OpenCode:gpt-oss-120b
brettweir force-pushed proofread-services from 40af27649c to 99a268f092 2026-07-13 02:07:30 +00:00 Compare
goroboro approved these changes 2026-07-13 17:01:32 +00:00
goroboro left a comment

Looks good to merge.

Looks good to merge.
@ -121,3 +121,3 @@
[TIP]
====
In case of connection issues there is most probably a SELinux configuration missing, in most case a "__name_connect access on the tcp_socket__" issue. Check the SELinux module in Cockpit. It helps you in detail to establish a local policy to fix it. It is easy, just a matter of 2 instructions.
In case of connection issues, there is probably a SELinux configuration missing, most likely a "__name_connect access on the tcp_socket__" issue. Check the SELinux module in Cockpit. It provides detailed guidance to fix it. It is easy—just a matter of two instructions.
Member

It is easy—just a matter of two instructions.

This smacks of AI speak... I would just change this to something like:
The SELinux module, in Cockpit, provides detailed guidance on how to fix this issues by running two simple instructions/commands.

It is easy—just a matter of two instructions. This smacks of AI speak... I would just change this to something like: The SELinux module, in Cockpit, provides detailed guidance on how to fix this issues by running two simple instructions/commands.
Author
Member

This is a good catch, I will ungrossify this.

This is a good catch, I will ungrossify this.
Author
Member

Oh, nevermind, I see now that I only added the em dash. The phrasing in question was part of the source document. While I think it should be addressed, it is out of scope for this pull request.

Oh, nevermind, I see now that I only added the em dash. The phrasing in question was part of the source document. While I think it should be addressed, it is out of scope for this pull request.
Author
Member

Tracking here server/tickets#209

Tracking here https://forge.fedoraproject.org/server/tickets/issues/209
brettweir marked this conversation as resolved
Owner

There is a conflict around line 143. Could you have a look into it, please?

There is a conflict around line 143. Could you have a look into it, please?
pboy merged commit 6f572aa137 into main 2026-07-15 20:53:23 +00:00
brettweir deleted branch proofread-services 2026-07-16 04:33:49 +00:00
Sign in to join this conversation.
No description provided.