Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

intro.9: minor changes #813

Closed
wants to merge 8 commits into from
Closed

intro.9: minor changes #813

wants to merge 8 commits into from

Conversation

grahamperrin
Copy link
Contributor

@grahamperrin grahamperrin commented Aug 6, 2023 •
edited

Correction: an (not and).

Other minor changes.

https://bugs.freebsd.org/270481

@grahamperrin
intro.9: minor changes
4e5a36d 
Correction: an (not and). 

Other minor changes
@grahamperrin
intro.9: reword the line for intr_event(9)
84ef825 
One long line was a little difficult to digest, at first.
@grahamperrin

This comment was marked as resolved.

Sign in to view
@grahamperrin grahamperrin marked this pull request as ready for review August 6, 2023 17:47
grahamperrin
grahamperrin commented Aug 6, 2023
View reviewed changes
share/man/man9/intro.9 Outdated Show resolved Hide resolved
share/man/man9/intro.9 Outdated Show resolved Hide resolved
mhorne
mhorne reviewed Aug 6, 2023
View reviewed changes
Copy link
Contributor

@mhorne mhorne left a comment

Choose a reason for hiding this comment

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

Thanks for your suggestions, see my replies. FYI, I do expect to make more changes to the document in the near future, so some bits are still subject to change.

share/man/man9/intro.9 Show resolved Hide resolved
share/man/man9/intro.9 Outdated Show resolved Hide resolved
share/man/man9/intro.9 Outdated Show resolved Hide resolved
share/man/man9/intro.9 Outdated Show resolved Hide resolved
share/man/man9/intro.9 Outdated Show resolved Hide resolved
share/man/man9/intro.9 Outdated Show resolved Hide resolved
share/man/man9/intro.9 Show resolved Hide resolved
share/man/man9/intro.9 Outdated Show resolved Hide resolved
share/man/man9/intro.9 Show resolved Hide resolved
share/man/man9/intro.9 Outdated Show resolved Hide resolved
@grahamperrin
intro(9): revert some suggested changes
0c4471a 
Resolve three conversations, and update the date.
@grahamperrin
intro(9): KASAN(9), KMSAN(9): reorder
b87b5a7 
Reorder the line for KASAN(9) and KMSAN(9).

Resolve freebsd#813 (comment)
@grahamperrin
intro(9): vnode(9): active voice
26786f1 
Resolve freebsd#813 (comment)
grahamperrin
grahamperrin commented Aug 7, 2023
View reviewed changes
share/man/man9/intro.9 Outdated Show resolved Hide resolved
freebsd-git pushed a commit that referenced this pull request Jan 31, 2024
@grahamperrin @mhorne
intro.9: minor changes
712fd5a 
A correction: 'and' -> 'an'. Plus, several tweaks for brevity or
clarity.

Reviewed by:	mhorne
MFC after:	3 days
Pull Request:	#813
@mhorne
Copy link
Contributor

mhorne commented Jan 31, 2024

Squashed, and pushed to main: 712fd5a.

Unfortunately I don't have permission to close the issue.

@bsdimp
Copy link
Member

bsdimp commented Feb 2, 2024

closed.

@bsdimp bsdimp closed this Feb 2, 2024
@bsdimp bsdimp added merged and removed ready labels Feb 2, 2024
@grahamperrin grahamperrin deleted the intro-9 branch February 4, 2024 00:08
freebsd-git pushed a commit that referenced this pull request Apr 18, 2024
@grahamperrin @mhorne
intro.9: minor changes
9050a43 
A correction: 'and' -> 'an'. Plus, several tweaks for brevity or
clarity.

Reviewed by:	mhorne
MFC after:	3 days
Pull Request:	#813

(cherry picked from commit 712fd5a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bsdimp bsdimp bsdimp left review comments

@mhorne mhorne mhorne approved these changes

Assignees
No one assigned
Labels
merged
Projects
None yet
Milestone
No milestone
3 participants
@grahamperrin @mhorne @bsdimp