diff options
| author | James Ennis <james.ennis@codethink.co.uk> | 2019-09-06 10:56:40 +0100 |
|---|---|---|
| committer | James Ennis <james.ennis@codethink.co.uk> | 2019-09-06 11:29:06 +0100 |
| commit | d69f2324e6978b41106ae4bc63e75339e4f3f98f (patch) | |
| tree | 291e263ae58e10b53c2146d31b58d2cf06b2909e /src/buildstream/_frontend/cli.py | |
| parent | 04c0d417c26629df34b846363c7e18ab34d98981 (diff) | |
| download | buildstream-d69f2324e6978b41106ae4bc63e75339e4f3f98f.tar.gz | |
Address no-else-raise pylint warning
In the latest released version of pylint (2.3.x), the
"no-else-raise" (R1720) warning has been introduced.
See:
http://pylint.pycqa.org/en/latest/whatsnew/changelog.html#what-s-new-in-pylint-2-3-0
There are many instances of this in our codebase, which this
patch addresses.
Diffstat (limited to 'src/buildstream/_frontend/cli.py')
| -rw-r--r-- | src/buildstream/_frontend/cli.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/buildstream/_frontend/cli.py b/src/buildstream/_frontend/cli.py index 3fad1664d..d45e3a578 100644 --- a/src/buildstream/_frontend/cli.py +++ b/src/buildstream/_frontend/cli.py @@ -629,9 +629,8 @@ def shell(app, element, sysroot, mount, isolate, build_, cli_buildtree, pull_, c if cli_buildtree == "always": # Exit early if it won't be possible to even fetch a buildtree with always option raise AppError("Artifact was created without buildtree, unable to launch shell with it") - else: - click.echo("WARNING: Artifact created without buildtree, shell will be loaded without it", - err=True) + click.echo("WARNING: Artifact created without buildtree, shell will be loaded without it", + err=True) else: # If the value has defaulted to ask and in non interactive mode, don't consider the buildtree, this # being the default behaviour of the command |