-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Add %O% option to make.bat template; update docs #6363
Conversation
Codecov Report
@@ Coverage Diff @@
## 2.0 #6363 +/- ##
=======================================
Coverage 83.64% 83.64%
=======================================
Files 278 278
Lines 40239 40239
Branches 5931 5931
=======================================
Hits 33657 33657
Misses 5250 5250
Partials 1332 1332 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add it to make.bat_t
also? It is still available.
The make.bat_t and make.bat.new_t templates now pass %O% as well as %SPHINXOPTS% to `sphinx-build`, in parity with Makefile.new_t. The sphinx-build documentation page was updated to explicitly note $(O) and %O% as shortcut variables. Closes sphinx-doc#6323, per the comment thread discussion.
6f9404d
to
0dbdae3
Compare
@tk0miya Sure! commit is revised. |
Thank you for quick work. Merged! |
Absolutely! Thanks for all your efforts contributing to the maintenance & improvement of the project! |
Subject
Add
%O%
option to make.bat templates.Feature or Bugfix
Detail
The
make.bat_t
andmake.bat.new_t
templates now pass%O%
as well as%SPHINXOPTS%
tosphinx-build
, in parity withMakefile.new_t
.The
sphinx-build
documentation page was updated to explicitly note$(O)
and%O%
as shortcut variables.Closes #6323, per the comment thread discussion there.
Purpose
sphinx-quickstart
Relates