Skip to content

Conversation

@jimcat8
Copy link
Contributor

@jimcat8 jimcat8 commented Aug 9, 2022

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)

<kbd>n</kbd>

* Go to the previous occurrence found:
* Find the last matching string:
Copy link
Contributor

Choose a reason for hiding this comment

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

@jimcat8 I think "previous" is a better word than "last" in this case. "last" intimates the very last occurrence of the string, rather than the one that occurred before. I'd be happy with "Find the previous matching string:"

`/[1-9]word` : search _1word_, _2word__`x`word_ where `x` is a number

* `^` : Search for a string starting the line.
* `^` : Search for lines that begin with a character.
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 fine, except it needs to be "character(s)" as it can be one or more.

@sspencerwire
Copy link
Contributor

@jimcat8, mostly good! If you would address the comments I've added and fix those things, I'll merge. Thank you!

@sspencerwire sspencerwire merged commit 3e1a091 into rocky-linux:main Aug 10, 2022
@sspencerwire
Copy link
Contributor

Thanks @jimcat8!

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