Skip to content

Commit

Permalink
Mark a few options that should not show up in ./pants help. (#5968)
Browse files Browse the repository at this point in the history
### Problem

`./pants help` contains core options that are useful to every pants command, and the vast majority of global options are hidden in order to keep it concise. A few non-essential options ended up there recently.

### Solution

Hide them.
  • Loading branch information
Stu Hood authored Jun 16, 2018
1 parent be8bfeb commit e749782
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/python/pants/option/global_options.py
Original file line number Diff line number Diff line change
Expand Up @@ -196,6 +196,7 @@ def register_bootstrap_options(cls, register):
register('--target-spec-file', type=list, dest='target_spec_files', daemon=False,
help='Read additional specs from this file, one per line')
register('--verify-config', type=bool, default=True, daemon=False,
advanced=True,
help='Verify that all config file values correspond to known options.')

register('--build-ignore', advanced=True, type=list, fromfile=True,
Expand All @@ -212,6 +213,7 @@ def register_bootstrap_options(cls, register):
register('--glob-expansion-failure', type=str,
choices=GlobMatchErrorBehavior.allowed_values,
default=GlobMatchErrorBehavior.default_option_value,
advanced=True,
help="Raise an exception if any targets declaring source files "
"fail to match any glob provided in the 'sources' argument.")

Expand Down Expand Up @@ -291,7 +293,8 @@ def register_bootstrap_options(cls, register):
# This option changes the parser behavior in a fundamental way (which currently invalidates
# all caches), and needs to be parsed out early, so we make it a bootstrap option.
register('--build-file-imports', choices=['allow', 'warn', 'error'], default='warn',
help='Whether to allow import statements in BUILD files')
advanced=True,
help='Whether to allow import statements in BUILD files')

register('--remote-store-server', advanced=True,
help='host:port of grpc server to use as remote execution file store.')
Expand All @@ -310,6 +313,7 @@ def register_bootstrap_options(cls, register):
# This should eventually deprecate the RunTracker worker count, which is used for legacy cache
# lookups via CacheSetup in TaskBase.
register('--process-execution-parallelism', type=int, default=multiprocessing.cpu_count(),
advanced=True,
help='Number of concurrent processes that may be executed either locally and remotely.')

@classmethod
Expand Down

0 comments on commit e749782

Please sign in to comment.