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

Smart Docs - Kernel Boot Parameters #373

Merged
merged 10 commits into from
Nov 27, 2024

Conversation

sounix000
Copy link
Contributor

@sounix000 sounix000 commented Sep 23, 2024

Description

Smart Docs - Kernel Boot Parameters.

This is a new PR because the old PR has become too corrupted with all the Git mess.

Are there any relevant issues/feature requests?

Is this (based on) existing content?

@sounix000 sounix000 force-pushed the ssarkar/new-pr-kernel-boot-parameters-smartdocs branch from 206f801 to 3558801 Compare October 25, 2024 09:41
@sounix000 sounix000 force-pushed the ssarkar/new-pr-kernel-boot-parameters-smartdocs branch from cbad41f to 76007fb Compare November 4, 2024 07:03
@sounix000 sounix000 force-pushed the ssarkar/new-pr-kernel-boot-parameters-smartdocs branch from 76007fb to 35ae6a7 Compare November 11, 2024 07:50
@sounix000 sounix000 force-pushed the ssarkar/new-pr-kernel-boot-parameters-smartdocs branch from dacab8a to bc04b62 Compare November 19, 2024 14:58
@sounix000 sounix000 marked this pull request as ready for review November 19, 2024 19:50
Copy link
Contributor

@taroth21 taroth21 left a comment

Choose a reason for hiding this comment

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

@sounix000 : Many thanks, this reads very well! Please don't be shocked about the number of comments - 95% of them apply to metadata changes (we have fine-tuned the templates and rules recently, I added suggestions for all of them).

Other than metadata changes only two minor nitpicks (and I spotted a few typos, which I ignored here because Daria will run her editorial review afterwards anyway and will catch them).

DC-kernel-boot-parameters-modify Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
concepts/kernel-boot-parameters-modify-introduction.xml Outdated Show resolved Hide resolved
tasks/kernel-boot-parameters-modify-troubleshoot.xml Outdated Show resolved Hide resolved
Copy link
Contributor

@taroth21 taroth21 left a comment

Choose a reason for hiding this comment

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

@sounix000 : Thanks a lot! Go ahead and merge :)

Copy link
Contributor

@dariavladykina dariavladykina left a comment

Choose a reason for hiding this comment

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

Hi! Please see some suggestions for this one. Thanks!

articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
tasks/kernel-boot-parameters-modify-troubleshoot.xml Outdated Show resolved Hide resolved
tasks/kernel-boot-parameters-modify-troubleshoot.xml Outdated Show resolved Hide resolved
tasks/kernel-boot-parameters-modify-troubleshoot.xml Outdated Show resolved Hide resolved
tasks/kernel-boot-parameters-modify-troubleshoot.xml Outdated Show resolved Hide resolved
glues/kernel-boot-parameters-modify-more-info.xml Outdated Show resolved Hide resolved
@sounix000 sounix000 force-pushed the ssarkar/new-pr-kernel-boot-parameters-smartdocs branch from b95b259 to 2762598 Compare November 26, 2024 05:28
Copy link
Contributor

@dariavladykina dariavladykina left a comment

Choose a reason for hiding this comment

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

Hi! Only some residual findings here. Thanks!

tasks/kernel-boot-parameters-modify-troubleshoot.xml Outdated Show resolved Hide resolved
concepts/kernel-boot-parameters-modify-introduction.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
Copy link
Contributor

@dariavladykina dariavladykina left a comment

Choose a reason for hiding this comment

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

Just a couple of corrections again, thanks!

articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
articles/kernel-boot-parameters-modify.asm.xml Outdated Show resolved Hide resolved
@sounix000 sounix000 merged commit f4ba4fc into main Nov 27, 2024
11 checks passed
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