-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
ENH: enable multiline signatures when too long. #11011
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
Merged
Merged
Changes from 106 commits
Commits
Show all changes
110 commits
Select commit
Hold shift + click to select a range
82f1b98
ENH: enable multiline signatures when too long.
TLouf 8d6dfba
Make parameter nodes inherit from sig classes
TLouf 581433b
Fix permalink for multiline Python signatures
TLouf d3ec233
Enable multiline sig for C and C++
TLouf 1229b66
Simplify `desc_parameterlist` into single class
TLouf ec713e9
Fix flake8 errors
TLouf 85ff493
Pass multiline as `desc_parameterlist` attribute
TLouf 8ca560d
Add type annotation
TLouf 9ec4698
Use new desc_parameterline instead of desc_content
TLouf b93ecf0
Support HTML writer
TLouf 3b110ce
Support text writer
TLouf 234acd6
Increment environment versions
TLouf 7c016c9
Better variable names
TLouf 5b8735d
Expand docstrings of new nodes' classes
TLouf 1ca68b0
Fix oversight of param line name change in C++
TLouf 8e198f3
Add newlines to make param list HTML prettier
TLouf 25ae353
Tweak EPUB's CSS to match HTML's
TLouf d6b2dac
Add tests for python domain
TLouf af77cad
Add tests for C domain
TLouf ea89180
Add tests for C++ domain
TLouf 5cc07d6
Add minimal implementation for man and texinfo
TLouf caff655
Python domain test cleanup
TLouf b71b06d
Make flake8 happy
TLouf 88c2e09
Make isort happy
TLouf 4258fd5
Add doc
TLouf 6e6261c
Make docslint happy
TLouf 9c52eae
Add types for `maximum_signature_line_length`
TLouf 2d9ad80
Fix typo
TLouf 0bc3b68
Add global `maximum_signature_line_length` config
TLouf 2c0da6a
Add tentative LaTeX implementation
TLouf bba2b51
Merge branch 'master' into signature-linebreaks
TLouf d8df099
Make docslint happy
TLouf d18f973
Merge branch 'master' into signature-linebreaks
jfbu d803e68
Cleanup latex style
TLouf 3753e7e
Rename new latex command for clarity
TLouf eb63da6
Force zero margins for dl tag
TLouf 53e882f
latex: avoid page break above first function argument
TLouf d7508a0
latex: match coding style of LaTeX macro files in for loop
TLouf b665647
latex: rename command to `pysigwithonelineperarg`
TLouf ada9d00
latex: avoid page break before dedented closing brace
TLouf ed7fac8
Update versionadded
TLouf 1475f5f
docs: fix linking to `single-line-signature`
TLouf 9b9e694
docs: better linking to `single-line-signature`
TLouf 691a780
Merge branch 'master' into signature-linebreaks
TLouf 4deb58c
Add trailing commas for ruff
TLouf 48d2ccb
Add trailing commas in tests
TLouf b065ab7
Add `desc_parameter_line` to doc
TLouf 8d42ac1
Merge branch 'master' into signature-linebreaks
AA-Turner 2b864fe
Update config docs
AA-Turner fa97a15
Update option docs
AA-Turner d787002
Move options to end
AA-Turner 707650f
Revert changes to the frozen HTML 4 writer
AA-Turner b1cb1b3
Revert ``is_multiline`` -> ```is_multi_line`` rename for ``desc_signa…
AA-Turner 8308742
``maximum_signature_line_length`` type to be int | None
AA-Turner e6ef3c2
Rename ``single-line-signature`` to ``single-line-{parameter,argument…
AA-Turner dfd0075
Refactor tests
AA-Turner bfab7ea
Set and use ``multi_line_parameter_list`` on node
AA-Turner 26551bc
Remove ``desc_parameter_line``
AA-Turner 9d03de0
options docs typo (macro -> function)
AA-Turner b3e5662
CHANGES entry
AA-Turner 592c86c
fixup! Remove ``desc_parameter_line``
AA-Turner 1af1a2e
Restore stripped whitespace
AA-Turner 69dba4b
fixup! (round 2) Remove ``desc_parameter_line``
AA-Turner 2918995
Remove unneeded py domain permalink logic
AA-Turner d58b63c
Restore cross-reference to single line option
TLouf b866b88
Add JS domain implentation
TLouf c6cf460
Mention @jfbu in CHANGES
TLouf b18c481
Unify directive option to `single-line-parameter-list`
TLouf 4b47e79
Fix directive hyperlink in CHANGES
TLouf a652d3e
Pass multiline bool to `_pseudo_parse_arglist`
TLouf bcba5f6
Handle bracket notation for optional params in HTML
TLouf 3243f9d
Rename `on_single_line` to `on_separate_line`
TLouf 93bdb8d
Fix latex for sigs starting with optional param
TLouf 417db1f
Cleanup of HTML
TLouf c4e9c62
Update LaTeX to handle optional parameters in one line per param context
jfbu 57004e3
Use some LaTeX hack to handle comma placement vs bracket
jfbu 88cfe5a
Mark latex strings as raw to ignore backslashes
TLouf 9ce3e34
Add trailing comma in expected HTML of tests
TLouf 3379337
Merge branch 'master' into signature-linebreaks
jfbu 98f1803
Move `and` operator to line start
TLouf ef7adde
Handle optionals in text writer
TLouf 1ad0981
Add `single-line-parameter-list` as JS object option
TLouf e36c811
Add essential JS tests
TLouf ea30441
Actually test for `multi_line_parameter_list` bool
TLouf 2d604b9
Add JS tests with `maximum_signature_line_length`
TLouf 71f5a8a
Remove (probably) forgotten c test template
TLouf e948563
Make test names consistent
TLouf d2b3459
Add newline after dd tags to make more readable HTML
TLouf bfc353e
Test for optional syntax in js and py domains
TLouf 0c54703
latex: handle comma placement in writer
TLouf a1b031f
latex: follow-up refactoring after TeX black magic removal
jfbu a0f5d9b
latex: tidy-up some comment in sphinxlatexstyletext.sty
jfbu a8a9be9
Fix comma placement when `required_params_left`
TLouf 3b34e30
Fix case of multiple params at same optional level
TLouf 2c1f83a
lint: remove extra line between method defs
TLouf 9c625f1
Conform test to new syntax
TLouf 5866fe6
Test for multiple params in optional level
TLouf 3bef7e4
Make HTML asserts more modular to ease debugging
TLouf 30041a6
Test xelatex PDF build
TLouf 8037a5b
Actually override xelatex' build conf
TLouf 2a85b9e
Merge branch 'master' into signature-linebreaks
jfbu 35424ea
Document the added \sphinxparam, \sphinxparamcomma...
jfbu db91327
Add a specific latex unit test
jfbu 1a44553
Remove blank line in latex.rst
jfbu 0c4fdb8
Simplify parametrize of latex build test
TLouf 36183f5
Add tests for text output
TLouf f2d1574
Bump versionadded in doc/latex.rst to match expected release number
jfbu 370cd0e
Bump again versionadded in doc/latex.rst to match expected release
jfbu 50ac9a6
Merge branch 'master' into signature-linebreaks
AA-Turner 585b2b9
misc updates (inc version bump)
AA-Turner File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.