-
-
Notifications
You must be signed in to change notification settings - Fork 61
Fix: typos #99
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
Fix: typos #99
Conversation
kouosi
commented
Oct 4, 2024
- Fix some spelling mistakes
- Remove some trailing space
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this PR. Left few comments (minor things).
Feel free to add yourself to the contributor list too.
@@ -73,7 +73,7 @@ That took a little more work than grub, but this can (and should) be automated a | |||
### Limine.cfg | |||
Similar to grub, limine also uses a config file. This config file has its own documentation, which is available in the limine repository. | |||
|
|||
Limine.cfg lists each boot entry as a title, followed by a series of key-value pairs. To boot our example from above using stivale 2, our config might look like the following: | |||
limine.cfg lists each boot entry as a title, followed by a series of key-value pairs. To boot our example from above using stivale 2, our config might look like the following: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it looks odd to start a sentence with lower case, maybe change to "The limine.cfg
file lists each ..."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If limine.cfg
is in backticks, does grub.cfg
also need to be?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, let's keep things consistent.
@@ -1,4 +1,4 @@ | |||
# Boot Protocols | |||
[#](#.md) Boot Protocols |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still see this odd syntax, [#](#.md)
, does it have any particular purpose? Otherwise i think it can be removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry my editor auto adds it for some reason
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm.
Do you want to add yourself (or being added) to the contributors list? Here https://github.com/dreamportdev/Osdev-Notes/blob/master/99_Appendices/I_Acknowledgments.md ?
I’ll decline this request as I only fixed a typo and don’t consider it a contribution |
Thanks! Thanks again for your help. |