diff options
author | Mike Frysinger <vapier@google.com> | 2019-11-11 05:40:22 -0500 |
---|---|---|
committer | Mike Frysinger <vapier@google.com> | 2019-11-12 03:44:39 +0000 |
commit | 3164d40e2247d42537aef8e80fa7e048e14bec9f (patch) | |
tree | 650cc33e3d5c4b39c3cc652e93495e47a170931b /repo | |
parent | f4545126197781beb03bb0fd47e7f24ce5af6ca8 (diff) | |
download | git-repo-3164d40e2247d42537aef8e80fa7e048e14bec9f.tar.gz |
use open context managers in more places
Use open() as a context manager to simplify the close logic and make
the code easier to read & understand. This is also more Pythonic.
Change-Id: I579d03cca86f99b2c6c6a1f557f6e5704e2515a7
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/244734
Reviewed-by: David Pursehouse <dpursehouse@collab.net>
Tested-by: Mike Frysinger <vapier@google.com>
Diffstat (limited to 'repo')
-rwxr-xr-x | repo | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -513,9 +513,8 @@ def SetupGnuPG(quiet): | |||
513 | sys.exit(1) | 513 | sys.exit(1) |
514 | print() | 514 | print() |
515 | 515 | ||
516 | fd = open(os.path.join(home_dot_repo, 'keyring-version'), 'w') | 516 | with open(os.path.join(home_dot_repo, 'keyring-version'), 'w') as fd: |
517 | fd.write('.'.join(map(str, KEYRING_VERSION)) + '\n') | 517 | fd.write('.'.join(map(str, KEYRING_VERSION)) + '\n') |
518 | fd.close() | ||
519 | return True | 518 | return True |
520 | 519 | ||
521 | 520 | ||