-
Notifications
You must be signed in to change notification settings - Fork 170
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
Some suggestions for minor text improvements in the docs #255
Conversation
This commit fixes a duplicated definite article 'the' in the last section and moves the corrected paragraph to the "Overview" section. This change is made because the paragraph contains the Wiki link that provides valuable information about libmetal internals, including a detailed section titled "OpenAMP Overview". It also addresses multiple format issues, including missing or duplicate spaces in the text, and corrects the line length to match the format used in README.md. Signed-off-by: Gustavo Romero <[email protected]>
Remove duplicated indefinite article in the text. Signed-off-by: Gustavo Romero <[email protected]>
@gromero Thanks for this change. This looks good to me. |
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.
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 the refactoring
Any reason to limit to 80 chars?, we accept up to 100 char per Line
Hi @arnopo . Just because most of the doc is already formatted as 80 chars. |
Yes mainly due to legacy 80 chars imposed by Linux Kernel coding rules. That said 80 chars is under 100 so you respect the library coding rules. So OK for me |
Just some suggestions for minor text improvements I have after going through the libmetal docs.
Cheers,
Gustavo