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

ci(actions): build documentation when a PR is opened #43

Merged
merged 2 commits into from
Mar 17, 2025

Conversation

bruxisma
Copy link
Member

After the recent deployment failure, checking to make sure documentation at least builds if any file under docs/ is built is worth the time we run the task for. Currently using the builtin paths argument, but might switch to the skip-duplicate-actions workflow so that we can mark this workflow as required.

Additionally, we're not doing anything with the output, we just want to make sure there aren't any build errors from vitepress at this time. If we move to starlight in the future, we'll most likely keep this same approach/workflow.

After the recent deployment failure, checking to make sure documentation at least builds if any file under `docs/` is built is worth the time we run the task for. Currently using the builtin `paths` argument, but might switch to the skip-duplicate-actions workflow so that we can mark this workflow as required.

Additionally, we're not doing anything with the output, we just want to make sure there aren't any build errors from vitepress at this time. If we move to starlight in the future, we'll most likely keep this same approach/workflow.

Signed-off-by: Izzy Muerte <[email protected]>
I can always add a check that the first word is lower-case in the
future.

Signed-off-by: Izzy Muerte <[email protected]>
@bruxisma bruxisma merged commit deeb60a into main Mar 17, 2025
6 checks passed
@bruxisma bruxisma deleted the actions/build-docs branch March 17, 2025 11:04
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.

1 participant