pantsbuild / pants / 5590
0%
main: 93%

LAST BUILD BRANCH: upgrade-hdr-histogram
DEFAULT BRANCH: main
Ran
Jobs 10
Files 335
Run time 9min
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
5590

push

travis-ci

Benjy 
Reimplement help formatting.

Using the argparse help formatter was no longer viable: we had
to customize it in so many ways it was easier to simply implement
our own. The new implementation:

- Supports subsystem args.
- Separates help info extraction from rendering, which allows
  more code reuse (and less hackiness) in builddict and bash
  completion generation tasks.
- Is better tested than the old code.
- Uses ANSI colors for readability.
- Sensibly handles recursively-registered options (it displays
  them in basic help on the outermost scope the option was
  registered on, but then only in advanced help in the inner
  scopes that inherited the registration via recursion).

This change also makes the implementations of builddict and bash
completion generation simpler and supportive of subsystem options.

This implementation requires knowledge of what a scope represents
(a task, goal, subsystem etc.) and also of who registered each
option on a scope. Note that these overlap but are not identical.
For example, a recursive global option is registered by the global
registrar on all task scopes.

This makes a lot of options code quite a lot simpler, so I took
the opportunity of this refactoring to simplify even further.
For example, we no longer have separate paths for registering
boolean and non-boolean options.

I also deleted a couple of unimportant builddict tests, which
relied more on incidental behavior than on intended properties
of that code.  It was going to be too much effort to get them
to pass (as TestOptions don't expose necessary methods, and it
would be overkill to add them just to support these trivial tests).

Testing Done:
CI passes here: https://travis-ci.org/pantsbuild/pants/builds/70451126

A previous run had one odd test failure which I'm currently investigating.

Reviewed at https://rbcommons.com/s/twitter/r/2458/

14116 of 20000 relevant lines covered (70.58%)

1.33 hits per line

Jobs
ID Job ID Ran Files Coverage
1 5590.1 (CI_FLAGS="-cjlpn 'Various pants self checks'") 0
Travis Job 5590.1
2 5590.2 (CI_FLAGS="-fkmsrcn -u 0/2 'Unit tests for pants and pants-plugins - shard 1'") 0
66.28
 Travis Job 5590.2
3 5590.3 (CI_FLAGS="-fkmsrcn -u 1/2 'Unit tests for pants and pants-plugins - shard 2'") 0
66.53
 Travis Job 5590.3
4 5590.4 (CI_FLAGS="-fkmsrcjlp 'Python contrib tests'") 0
Travis Job 5590.4
5 5590.5 (CI_FLAGS="-fkmsrjlpn -i 0/6 'Python integration tests for pants - shard 1'") 0
Travis Job 5590.5
6 5590.6 (CI_FLAGS="-fkmsrjlpn -i 1/6 'Python integration tests for pants - shard 2'") 0
Travis Job 5590.6
7 5590.7 (CI_FLAGS="-fkmsrjlpn -i 2/6 'Python integration tests for pants - shard 3'") 0
Travis Job 5590.7
8 5590.8 (CI_FLAGS="-fkmsrjlpn -i 3/6 'Python integration tests for pants - shard 4'") 0
Travis Job 5590.8
9 5590.9 (CI_FLAGS="-fkmsrjlpn -i 4/6 'Python integration tests for pants - shard 5'") 0
Travis Job 5590.9
10 5590.10 (CI_FLAGS="-fkmsrjlpn -i 5/6 'Python integration tests for pants - shard 6'") 0
Travis Job 5590.10
Source Files on build 5590
Detailed source file information is not available for this build.