pantsbuild / pants / 8872
0%
main: 93%

LAST BUILD BRANCH: deps/opendal-0.57
DEFAULT BRANCH: main
Ran
Jobs 7
Files 0
Run time –
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
8872

push

travis-ci

stuhood 
Adding support for multiline param help descriptions in Pants BUILD Dictionary

In response to a user reported error that target parameter help appeared incomplete, added support for parameter help strings that span multiple lines.

Prior to this change, get_arg_descriptions_from_docstring did not suppport multiline descriptions, as was noted in the comments.  However, there are numerous instances in which existing parameter help spans multiple lines, and there's nothing enforcing the single-line description convention.  In addition, especially for parameters that have longer names, it may be difficult to include meaningful descriptions that span only a single line.  Thus, it seemed best to go ahead and add support for multiline descriptions.

Adding support for multiline descriptions also fixed the fact that no help was appearing in the Pants BUILD Dictionary for parameters where the help string included :param name: on one line and the description on the next line.   This can be seen in the help for the provides parameter of contrib_plugin (screenshots attached).

As seen in the screenshots for android_binary below, adding support for mulitline descriptions altered the formatting of the tables containing those descriptions.  In particular, the parameter names sometimes wrap, and the description text is aligned in the middle of the cell.  If these changes go in, I'll open another github issue on that, given my very limited css skills.

Testing Done:
https://travis-ci.org/pantsbuild/pants/builds/106130379

Bugs closed: 2848

Reviewed at https://rbcommons.com/s/twitter/r/3399/
Jobs
ID Job ID Ran Files Coverage
1 8872.1 (CI_FLAGS="-cjlpn 'Various pants self checks'") 0
Travis Job 8872.1
4 8872.4 (CI_FLAGS="-fkmsrcjlp 'Python contrib tests'") 0
Travis Job 8872.4
6 8872.6 (CI_FLAGS="-fkmsrjlpn -i 1/6 'Python integration tests for pants - shard 2'") 0
Travis Job 8872.6
7 8872.7 (CI_FLAGS="-fkmsrjlpn -i 2/6 'Python integration tests for pants - shard 3'") 0
Travis Job 8872.7
8 8872.8 (CI_FLAGS="-fkmsrjlpn -i 3/6 'Python integration tests for pants - shard 4'") 0
Travis Job 8872.8
9 8872.9 (CI_FLAGS="-fkmsrjlpn -i 4/6 'Python integration tests for pants - shard 5'") 0
Travis Job 8872.9
10 8872.10 (CI_FLAGS="-fkmsrjlpn -i 5/6 'Python integration tests for pants - shard 6'") 0
Travis Job 8872.10
Source Files on build 8872
Detailed source file information is not available for this build.