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

Update example documentation #23

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

wiba-nordic
Copy link

Updates the Example Application's document to use
the sphinx-ncs-theme v0.7.4, which was the latest version I could get to easily work.

Also adds some example pages and documentation guidance.

carlescufi and others added 5 commits May 27, 2024 15:32
Adapt original Zephyr repo to NCS, including the manifest and the doc.

Signed-off-by: Carles Cufi <[email protected]>
Signed-off-by: Markus Tacker <[email protected]>
Signed-off-by: Pekka Niskanen <[email protected]>
Signed-off-by: Jamie McCrae <[email protected]>
Signed-off-by: Vinayak Kariappa Chettimada <[email protected]>
Signed-off-by: Gerard Marull-Paretas <[email protected]>
Add an action which builds and tests application using ncs-docker image.
nrf-squash! [nrf noup] treewide: Adapt to NCS

Uses sysbuild when building the application using twister

Signed-off-by: Jamie McCrae <[email protected]>
Since the migration to Zoomin, this is no longer possible. Remove the
functionality until a workaround is found.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
Updates the Example Application's document to use
the sphinx-ncs-theme v0.7.4, which was the latest version
I could get to easily work.

Also adds some example pages and documentation guidance.

Signed-off-by: Wille Backman <[email protected]>
@wiba-nordic wiba-nordic added the documentation Improvements or additions to documentation label Sep 25, 2024
@wiba-nordic
Copy link
Author

No idea idea how to fix the build error. The documentation builds locally.

Copy link
Member

@gmarull gmarull left a comment

Choose a reason for hiding this comment

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

most changes seem unnecessary, also, we should not recommend any theme, this is for custom applications, why will they want Nordic logo?

@wbober
Copy link

wbober commented Sep 25, 2024

most changes seem unnecessary, also, we should not recommend any theme, this is for custom applications, why will they want Nordic logo?

Nordic theme is useful for Nordic teams that move their products to add-ons. I would like it to be available. Can we make it, so it is one of the available options (default or Nordic)?

Edit:
I see that it is just a simple change in the config file. I think it would be sufficient to just document how to change the theme and what are the options available.

@wiba-nordic
Copy link
Author

wiba-nordic commented Sep 25, 2024

In addition to what Wojciech said, the Example Application is still a Nordic offering, so it is good for it to have Nordic styling and quality. I do agree that it would probably be good to document how to change the theme, in which case we can leave the Alabaster theme as a commented-out option in the file.

The changed content is likewise good for any Nordic projects, and makes it easier for outside contributors to create documentation that is at least somewhat consistent with Nordic documentation. This, in turn, is good for end users when the add-ons are consistently documented.

@gmarull
Copy link
Member

gmarull commented Sep 25, 2024

Note: Nordic theme is no longer maintained, as we've moved to Zoomin. So at some point it will likely break.

@wiba-nordic
Copy link
Author

wiba-nordic commented Sep 25, 2024

How much of the theme is the CSS (which shouldn't break), and how much is Breathe/Sphinx functionality?

@gmarull
Copy link
Member

gmarull commented Sep 25, 2024

How much of the theme is the CSS (which shouldn't break), and how much is Breathe/Sphinx functionality?

It depends on rtd theme, and has quite a few tweaks on top, like supporting tabs, version dropdown, etc. I'd not recommend it for external usage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants