Skip to content

Commit 8d80da8

Browse files
committed
Smol cleanups
Removing the `exit(1)` after failing. This shouldn't have an impact since we've already failed and shouldn't have hit that. I may have kept it from earlier debugging. Replace `raise e` with just `raise`. It removes one level from the stack trace where the actual `raise` takes place. The place we're actually interested in is where the failure actually takes place.
1 parent 73d2ac1 commit 8d80da8

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

utils/update_checkout/update_checkout/update_checkout.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -132,8 +132,7 @@ def get_branch_for_repo(config, repo_name, scheme_name, scheme_map,
132132
.format(pr_id, repo_branch), "--tags"], echo=True)
133133
except KeyError as e:
134134
print(f"Failed to look up {repo_name} in scheme", file=sys.stderr)
135-
raise e
136-
exit(1)
135+
raise
137136
return repo_branch, cross_repo
138137

139138

0 commit comments

Comments
 (0)