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

Fix documentation, including library sample links #2453

Merged
merged 6 commits into from
Dec 24, 2021

Conversation

mikee47
Copy link
Contributor

@mikee47 mikee47 commented Dec 21, 2021

This PR addresses some issues raised in the documentation build log, such as missing references, ill-formed tables, etc.

Some documentation sections have been extended.

:sample: links in libraries have been fixed. It was previously assumed that all samples were located in the main 'samples' directory, so links within library documentation to contained samples did not work. They're now a bit smarter so will work logically when used within Components. They can also direct to a specific Component such as :sample:`CS5460/generic` .

:source: links have also been fixed so they can be used within Components. Links starting with '/' refer to Sming tree, otherwise relative to Component root.

See https://smingdev.readthedocs.io/en/fix-documentation/ for test documentation build.

todo

  • Fix incorrect use of doxygen @name tag

@slaff slaff added this to the 4.5.0 milestone Dec 22, 2021
@slaff
Copy link
Contributor

slaff commented Dec 22, 2021

@mikee47 Wow, so far this PR looks really good! Thank you Mikee!

@slaff slaff mentioned this pull request Dec 22, 2021
5 tasks
… document location

i.e. if used within a library then first look  in library for a sample with the given name.
If not found, use default samples location.
@mikee47 mikee47 force-pushed the fix/documentation branch 3 times, most recently from 06afa3a to 92d8303 Compare December 22, 2021 22:18
- Links within Components are relative to the Component root directory
- File links always refer to the repository, local links are no longer produced.
- Links starting with '/' refer to Sming tree, not Component
@slaff slaff merged commit a8b4c24 into SmingHub:develop Dec 24, 2021
@mikee47 mikee47 deleted the fix/documentation branch January 15, 2023 19:36
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.

2 participants