Skip to content

Implement a barebones CI for setup for the user docs. #24

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

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

albu-diku
Copy link
Contributor

Add a "build-test" Makefile target which executes a sphinx docs build with warnings turned into errors. Use this new target within a GitHub workflow such that the ability to build the docs is verified on each commit.

@albu-diku albu-diku force-pushed the test/docs-build-ci branch 2 times, most recently from 06b39e1 to 485a128 Compare February 3, 2025 11:40
@albu-diku
Copy link
Contributor Author

This currently fails because the docs do no build cleanly as-is on the main branch. I have opened a pull request to fix the issue: #22. cc @aputtu @SebastianPrehn

@SebastianPrehn
Copy link
Contributor

Unable to build due to https://github.com/ucphhpc/migrid-user-docs/blob/main/docs/sections/shared_content/ucph_login.rst not being included in the tocs. @aputtu Is this meant to replace something, or was it a file to test some features? If the latter, we should probably find some way to store them so that they do not generate warnings

Add a "build-test" Makefile target which executes a sphinx docs build with
warnings turned into errors. Use this new target within a GitHub workflow
such that the ability to build the docs is verified on each commit.
@SebastianPrehn SebastianPrehn merged commit 6d5d744 into main Feb 6, 2025
1 check passed
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