-
Notifications
You must be signed in to change notification settings - Fork 27
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
Cleanup of factory reference manual section #628
Conversation
Docs for bc84a46 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2484/docs/artifacts/html/index.html |
The image |
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.
Only few suggestion
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.
review suggestions
Docs for 40bb1ab are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2485/docs/artifacts/html/index.html |
40bb1ab
to
34ea46b
Compare
Docs for 34ea46b are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2486/docs/artifacts/html/index.html |
Docs for 55be7fd are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2487/docs/artifacts/html/index.html |
Style guide lines applied. Removed link to the Yocto Project layers section which is part of the LmP reference manual; this was done to make navigation less confusing, and to keep sections focused. The section on Factory Sources was placed before the Factory Definition, for better explanatory flow. QA steps: Ran linter and addressed issues where possible/appropriate. Checked html output in browser, no obvious rendering issues discovered. Ran linkcheck. This commit addresses FFTK-2790 This commit addresses FFTK-1880 This commit applies to FFTK-2732 This commit applies to FFTK-2730 This commit applies to FFTK-988 Signed-off-by: Katrina Prosise <[email protected]>
55be7fd
to
5c97cd7
Compare
Docs for 5c97cd7 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2488/docs/artifacts/html/index.html |
Style guide lines applied. Removed link to the Yocto Project layers section which is part of the LmP reference manual; this was done to make navigation less confusing, and to keep sections focused.
The section on Factory Sources was placed before the Factory Definition, for better explanatory flow.
QA steps: Ran linter and addressed issues where possible/appropriate. Checked html output in browser, no obvious rendering issues discovered. Ran linkcheck.
This commit addresses FFTK-2790
This commit addresses FFTK-1880
This commit applies to FFTK-2732
This commit applies to FFTK-2730
This commit applies to FFTK-988
Readiness
Overview
Why merge this PR? What does it solve?
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."
make linkcheck
.-s, --signoff
).-S, --gpg-sign
).Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.