Skip to content

Conversation

@jimcat8
Copy link
Contributor

@jimcat8 jimcat8 commented Oct 3, 2022

…ed and intuitive

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)

@alemorvan
Copy link
Contributor

These changes are very interesting.
However, beware of the pedagogical progression of the document.
All the notions you introduce are seen in the rest of the document.
You can't say everything at the same time, otherwise readers will find themselves drowning in information.

@jimcat8
Copy link
Contributor Author

jimcat8 commented Oct 3, 2022

These changes are very interesting. However, beware of the pedagogical progression of the document. All the notions you introduce are seen in the rest of the document. You can't say everything at the same time, otherwise readers will find themselves drowning in information.

My personal view is to emphasize these documents repeatedly to deepen the impression of beginners. Since classification is a books, it shouldn't be so bad to write more detailed information! Of course, I have no right to decide whether to merge or not.

@alemorvan
Copy link
Contributor

You are not wrong!
I add my thumb.

@sspencerwire sspencerwire merged commit 20b8789 into rocky-linux:main Oct 4, 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.

3 participants