Skip to content

Conversation

@bkeryan
Copy link
Collaborator

@bkeryan bkeryan commented May 8, 2025

What does this Pull Request accomplish?

Copy Sphinx config from nitypes

Why should this Pull Request be merged?

Building docs during development helps write better docstrings.

What testing has been done?

poetry install --with docs
poetry run sphinx-build docs docs/_build -b html -W --keep-going

@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2025

Test Results

10 files  ±0  10 suites  ±0   6s ⏱️ ±0s
 6 tests ±0   6 ✅ ±0  0 💤 ±0  0 ❌ ±0 
60 runs  ±0  60 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit d6805c3. ± Comparison against base commit 3c7d248.

♻️ This comment has been updated with latest results.

@bkeryan
Copy link
Collaborator Author

bkeryan commented May 8, 2025

FYI, GitHub is waiting for status to be reported on "Run CI / Check / Check nipanel" because I renamed it to "Run CI / Check nipanel / Check nipanel". I will update the required checks in the GitHub admin settings before merging this PR.

@csjall csjall removed the request for review from mikeprosserni May 8, 2025 14:07
@bkeryan bkeryan force-pushed the users/bkeryan/sphinx-docs branch from 454225c to d6805c3 Compare May 8, 2025 14:21
@bkeryan bkeryan merged commit e9622d7 into main May 8, 2025
14 checks passed
@bkeryan bkeryan deleted the users/bkeryan/sphinx-docs branch May 8, 2025 14:25
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.

3 participants