Eliminate unnecessary URL options from the command line. #530
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.
This pull request removes the
--url
option from various CLI commands and updates the documentation accordingly. The changes ensure that the client now relies solely on the--backend
option to determine the ServiceX server.The --url option was never used. It's easy enough to just add a new entry to .servicex and this option was cluttering up the code
Key changes include:
Documentation Updates:
docs/command_line.rst
: Removed the-u
/--url
option from the list of common command line arguments and updated the description to reflect that only the--backend
option is required.Code Changes:
servicex/app/cli_options.py
: Removed theurl_cli_option
definition.servicex/app/codegen.py
: Removed theurl
parameter from theflush
andlist
commands and their respectiveServiceXClient
instantiations. [1] [2]servicex/app/datasets.py
: Removed theurl
parameter from thelist
,get
, anddelete
commands and their respectiveServiceXClient
instantiations. [1] [2] [3] [4] [5]servicex/app/transforms.py
: Removed theurl
parameter from thelist
,files
,download
, andlogs
commands and their respectiveServiceXClient
instantiations. [1] [2] [3] [4] [5] [6] [7] [8] [9]