Skip to content

Commit a0578e6

Browse files
committed
Merge pull request #940 from deis/more-cli-return-codes
fix(client): return 1 on two more error cases
2 parents adfcdf4 + 48cd94f commit a0578e6

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

client/deis.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@
5050
import json
5151
import locale
5252
import os.path
53-
import random
5453
import re
5554
import subprocess
5655
import sys
@@ -639,7 +638,7 @@ def auth_register(self, args):
639638
print('Login failed')
640639
else:
641640
print('Registration failed', response.content)
642-
return False
641+
sys.exit(1)
643642
return True
644643

645644
def auth_cancel(self, args):
@@ -651,7 +650,7 @@ def auth_cancel(self, args):
651650
controller = self._settings.get('controller')
652651
if not controller:
653652
print('Not logged in to a Deis controller')
654-
return False
653+
sys.exit(1)
655654
print('Please log in again in order to cancel this account')
656655
username = self.auth_login({'<controller>': controller})
657656
if username:

0 commit comments

Comments
 (0)