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

Simplify documentation workflow #449

Merged
merged 1 commit into from
Apr 11, 2024
Merged

Conversation

jfrost-mo
Copy link
Member

@jfrost-mo jfrost-mo commented Mar 26, 2024

Using sphinx directly rather than tox is faster, and removes a layer of abstraction to keep things more understandable. This means that tox is now only used interactively.

@jfrost-mo jfrost-mo marked this pull request as ready for review March 26, 2024 10:28
Copy link
Contributor

github-actions bot commented Mar 26, 2024

Coverage

@jfrost-mo jfrost-mo marked this pull request as draft March 26, 2024 10:31
@jfrost-mo jfrost-mo changed the title Directly use sphinx rather than tox in documentation workflow Simplify documentation workflow and use regular install for CI tests Mar 26, 2024
@jfrost-mo jfrost-mo added documentation Improvements or additions to documentation cleanup Non-functional improvement and removed documentation Improvements or additions to documentation labels Mar 28, 2024
This is faster, and removes a layer of abstraction to keep things more
understandable.

This means that tox is now only used interactively.
@jfrost-mo jfrost-mo force-pushed the documentation_workflow_drop_tox branch from fb794b7 to 2dc4088 Compare March 28, 2024 15:04
@jfrost-mo jfrost-mo changed the title Simplify documentation workflow and use regular install for CI tests Simplify documentation workflow Mar 28, 2024
@jfrost-mo jfrost-mo marked this pull request as ready for review March 28, 2024 15:07
@jfrost-mo
Copy link
Member Author

Boring technical change, not affecting the actual CSET code, so merging without review.

@jfrost-mo jfrost-mo merged commit b5a7874 into main Apr 11, 2024
8 checks passed
@jfrost-mo jfrost-mo deleted the documentation_workflow_drop_tox branch April 11, 2024 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Non-functional improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant