Skip to content

Conversation

@sspencerwire
Copy link
Contributor

@sspencerwire sspencerwire commented Mar 19, 2022

  • checked instructions and document for readability
  • checked for spelling issues and corrected
  • checked grammar usage and sentence structure - there may be some edits
    needed
  • checked all of the instructions for technical correctness, including
    those dealing with the firewalld instructions
  • checked the functions of Pound, both with and without the firewall
    services running
  • provided screenshots where needed

Author checklist (Completed by original Author)

  • Contribution a good fit for the Rocky project? Title and Author MetaTags inserted ?
  • Is this a non-English contribution?
  • If applicable, steps and instructions have been tested to work on a real system
  • Did you perform an initial self-review to fix basic typos and grammatical correctness

Rocky Documentation checklist (Completed by Rocky team)

  • 1st Pass (Check that document is good fit for project and author checklist completed)
  • 2nd Pass (Technical Review - check for technical correctness)
  • 3rd Pass (Basic Editorial Review)
  • 4th Pass (Detailed Editorial Review and Peer Review)
  • Final pass/approval (Final Review)

* checked instructions and document for readability
* checked for spelling issues and corrected
* checked grammar usage and sentence structure - there may be some edits
needed
* checked all of the instructions for technical correctness, including
those dealing with the firewalld instructions
* checked the functions of Pound, both with and without the firewall
services running
* provided screenshots where needed
@sspencerwire
Copy link
Contributor Author

I think this is pretty clean, but it may need some editing.

EzequielBruni and others added 2 commits March 23, 2022 17:33
A couple of typos and small changes. Thanks @EzequielB for your review!
@sspencerwire sspencerwire merged commit 85541c0 into rocky-linux:main Mar 24, 2022
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.

2 participants