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.

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

Information on kernel configuration and selection #53

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

rwatson
Copy link
Member

@rwatson rwatson commented Dec 23, 2023

This pull request includes some basic information on kernel configuration, selection, etc. It’s quite 23.11-specific, and I’ve tried to note that throughout.

Copy link
Member

@brooksdavis brooksdavis left a comment

Choose a reason for hiding this comment

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

I'm not sure about calling out 23.11 given that we're now publishing the guide per-release. I guess it should prompt us to look at the section next time.

src/features/kernel.md Outdated Show resolved Hide resolved
src/features/kernel.md Outdated Show resolved Hide resolved
src/features/kernel.md Outdated Show resolved Hide resolved
@rwatson
Copy link
Member Author

rwatson commented Jan 6, 2024

I'm not sure about calling out 23.11 given that we're now publishing the guide per-release. I guess it should prompt us to look at the section next time.

I’ve now reduced the extent to which this is true, but left one reference in a subheading.

@rwatson rwatson requested a review from brooksdavis January 7, 2024 17:03
Copy link
Member

@kwitaszczyk kwitaszczyk left a comment

Choose a reason for hiding this comment

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

Looks good. There is one typo and I've added one suggestion but feel free to ignore it if it makes the section too long.

src/features/kernel.md Outdated Show resolved Hide resolved
src/features/kernel.md Show resolved Hide resolved
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.

4 participants