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

Starting point for sphinx-based user documentation #74

Merged
merged 11 commits into from
Mar 30, 2021

Conversation

ejoerns
Copy link
Member

@ejoerns ejoerns commented Jan 27, 2021

This adds a basic skeleton for rauc-hawkbit-updater user documentation based on sphinx.

The tests are extended to test for doc (sphinx + doxygen) building, too.

Note that this also contains a fix for the current CI that fails to find changed debian repos.

The next step would then be to register a readthedocs subdomain for this, to allow browsing the documentation online.

Copy link
Member

@Bastian-Krause Bastian-Krause left a comment

Choose a reason for hiding this comment

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

Thanks!

docs/extensions/lexer/DtsLexer.pyc shouldn't be under version control. Maybe add this to .gitignore?

.github/workflows/tests.yml Outdated Show resolved Hide resolved
CHANGES Outdated Show resolved Hide resolved
docs/conf.py Outdated Show resolved Hide resolved
docs/contributing.rst Outdated Show resolved Hide resolved
docs/contributing.rst Outdated Show resolved Hide resolved
docs/reference.rst Outdated Show resolved Hide resolved
docs/reference.rst Outdated Show resolved Hide resolved
docs/reference.rst Outdated Show resolved Hide resolved
docs/reference.rst Outdated Show resolved Hide resolved
docs/reference.rst Outdated Show resolved Hide resolved
@ejoerns
Copy link
Member Author

ejoerns commented Feb 4, 2021

I've rebased this on master so the CI fix is now not part of this anymore.

Additionally, I structured it in a way that it starts from a vanilla sphinx-quickstart-generated configuration.

The theme is now changed to use readthedocs which requires an additional package but will have the same appearance in local builds as it would have for readthedocs builds.

docs/using.rst Outdated Show resolved Hide resolved
docs/using.rst Outdated Show resolved Hide resolved
docs/using.rst Outdated Show resolved Hide resolved
docs/using.rst Outdated Show resolved Hide resolved
@ejoerns
Copy link
Member Author

ejoerns commented Mar 5, 2021

@Bastian-Krause I should have finally addressed your review comments now.

Signed-off-by: Enrico Joerns <ejo@pengutronix.de>
As git does not care for empty directories, these directories created by
sphinx-quickstart will not exist when cloning the repo. Thus we simply
comment them out as long as we do not have any content for them.

Signed-off-by: Enrico Joerns <ejo@pengutronix.de>
Signed-off-by: Enrico Joerns <ejo@pengutronix.de>
Signed-off-by: Enrico Joerns <ejo@pengutronix.de>
Make sphinx warnings fatal for CI

Signed-off-by: Enrico Joerns <ejo@pengutronix.de>
Signed-off-by: Enrico Joerns <ejo@pengutronix.de>
The structure is derived from the parent 'rauc' project.

This also adds a `Developer's Certificate of Origin`

Signed-off-by: Enrico Joerns <ejo@pengutronix.de>
Signed-off-by: Enrico Joerns <ejo@pengutronix.de>
@ejoerns
Copy link
Member Author

ejoerns commented Mar 5, 2021

Rebased to benefit from testing fix #76.

Bastian-Krause
Bastian-Krause previously approved these changes Mar 5, 2021
Copy link
Member

@Bastian-Krause Bastian-Krause left a comment

Choose a reason for hiding this comment

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

Nice!

@Bastian-Krause
Copy link
Member

@ejoerns Pushed some small fixups, could you have a look and squash them if they look good?

ejoerns and others added 3 commits March 16, 2021 20:21
Signed-off-by: Enrico Joerns <ejo@pengutronix.de>
This adds the recently introduced "post_update_reboot" option with an
explanation what it does and what it doesn't.

Signed-off-by: Bastian Krause <bst@pengutronix.de>
Signed-off-by: Enrico Joerns <ejo@pengutronix.de>
@ejoerns
Copy link
Member Author

ejoerns commented Mar 16, 2021

@Bastian-Krause Thanks, they look quite reasonable! I've fixed up all but the one that adds new text and already got a SoB anyway. I left this as a stand-alone commit.

@ejoerns ejoerns merged commit eabb20d into rauc:master Mar 30, 2021
@ejoerns ejoerns deleted the topic/sphinx-doc branch March 30, 2021 10:05
@ejoerns
Copy link
Member Author

ejoerns commented Mar 30, 2021

With this merged, we now have https://rauc-hawkbit-updater.readthedocs.io/

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

Successfully merging this pull request may close these issues.

2 participants