search

neuroinformatics-unit / NeuroBlueprint

Lightweight data specification for systems neuroscience, inspired by BIDS.
http://neuroblueprint.neuroinformatics.dev/
Creative Commons Attribution 4.0 International
17 stars 1 forks source link

updated sphinx docs workflow #18

Closed niksirbi closed 1 year ago

niksirbi commented 1 year ago

Updated to new docs build workflow, which is now part of our v2 actions release

lauraporta commented 1 year ago

Comment from codereview.gpt

Based on the provided patch and commit message, here is my feedback as a code reviewer:

The commit message provides clear information about what the changes are meant to achieve. It seems that the code changes aim to update the workflow for building and deploying Sphinx documentation to GitHub pages. One workflow file, check_docs.yml, has been deleted, and a new workflow file, docs_build_and_deploy.yml, has been created. Another workflow file, publish_docs.yml, has also been deleted. The new workflow file includes steps to build and deploy the documentation to GitHub pages. The on section of the new workflow file is configured to generate the documentation on all merges to main, all pull requests, or by manual workflow dispatch. There are no bugs or issues that I can identify from the patch provided. Overall, these changes seem to be positive improvements to the workflow for building and deploying Sphinx documentation to GitHub pages.