Skip to content

Conversation

@bkeryan
Copy link
Collaborator

@bkeryan bkeryan commented May 8, 2025

What does this Pull Request accomplish?

Remove --keep-going from the Sphinx command line

Why should this Pull Request be merged?

--keep-going is always enabled in Sphinx 8.1+: https://www.sphinx-doc.org/en/master/man/sphinx-build.html#cmdoption-sphinx-build-keep-going

What testing has been done?

PR build

@bkeryan bkeryan requested a review from csjall as a code owner May 8, 2025 22:15
@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2025

Test Results

   10 files  ±0     10 suites  ±0   25s ⏱️ ±0s
  532 tests ±0    532 ✅ ±0  0 💤 ±0  0 ❌ ±0 
5 320 runs  ±0  5 320 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit cc55e44. ± Comparison against base commit 4dbdf64.

@bkeryan bkeryan merged commit b17f855 into main May 9, 2025
14 checks passed
@bkeryan bkeryan deleted the users/bkeryan/sphinx-keep-going branch May 9, 2025 15:13
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