]> git.scottworley.com Git - git-cache/commitdiff
New pylint likes explicit encodings
authorScott Worley <scottworley@scottworley.com>
Wed, 23 Nov 2022 08:33:37 +0000 (00:33 -0800)
committerScott Worley <scottworley@scottworley.com>
Wed, 23 Nov 2022 08:33:37 +0000 (00:33 -0800)
git_cache.py
test_git_cache.py

index 75f7b2689b8b7f12313aedc60754c349b53fcdd3..c7bcdd215e2cf52bda04483ecb267d0b72ade596 100644 (file)
@@ -85,7 +85,7 @@ def _read_fetch_log(repo: Repo) -> Iterator[_LogEntry]:
     filename = _log_filename(repo)
     if not os.path.exists(filename):
         return
     filename = _log_filename(repo)
     if not os.path.exists(filename):
         return
-    with open(filename, 'r') as f:
+    with open(filename, 'r', encoding='utf-8') as f:
         for line in f:
             _, _, rev, ref = line.strip().split(maxsplit=3)
             yield _LogEntry(ref, rev)
         for line in f:
             _, _, rev, ref = line.strip().split(maxsplit=3)
             yield _LogEntry(ref, rev)
@@ -107,7 +107,7 @@ def _log_fetch(repo: Repo, ref: Ref, rev: Rev, force: bool = False) -> None:
             verify_ancestry(repo, rev, prev_rev)
     filename = _log_filename(repo)
     os.makedirs(os.path.dirname(filename), exist_ok=True)
             verify_ancestry(repo, rev, prev_rev)
     filename = _log_filename(repo)
     os.makedirs(os.path.dirname(filename), exist_ok=True)
-    with open(filename, 'a') as f:
+    with open(filename, 'a', encoding='utf-8') as f:
         f.write(
             f'{time.strftime("%Y-%m%d-%H:%M:%S%z")} '
             f'{"FORCEDFETCH" if force else "fetch"} {rev} {ref}\n'
         f.write(
             f'{time.strftime("%Y-%m%d-%H:%M:%S%z")} '
             f'{"FORCEDFETCH" if force else "fetch"} {rev} {ref}\n'
@@ -189,7 +189,7 @@ def fetch(repo: Repo, ref: Ref, force: bool = False) -> Tuple[Path, Rev]:
     logging.debug('Fetching ref "%s" from %s', ref, repo)
     _git_fetch(cachedir, repo, ref, force=force)
 
     logging.debug('Fetching ref "%s" from %s', ref, repo)
     _git_fetch(cachedir, repo, ref, force=force)
 
-    with open(os.path.join(cachedir, 'refs', 'heads', ref)) as rev_file:
+    with open(os.path.join(cachedir, 'refs', 'heads', ref), encoding='utf-8') as rev_file:
         rev = Rev(rev_file.read(999).strip())
     verify_ancestry(repo, ref, rev, force=force)
     _log_fetch(repo, ref, rev, force=force)
         rev = Rev(rev_file.read(999).strip())
     verify_ancestry(repo, ref, rev, force=force)
     _log_fetch(repo, ref, rev, force=force)
index cc3cf3533715ed16e87198e9fb426215b13e5a7b..e97a788cd726b307791c95e55a1a00a90d4c1939 100644 (file)
@@ -27,7 +27,7 @@ def _commit_file(
         filename: str,
         contents: str,
         commit_message: str) -> None:
         filename: str,
         contents: str,
         commit_message: str) -> None:
-    with open(os.path.join(directory, filename), 'w') as f:
+    with open(os.path.join(directory, filename), 'w', encoding='utf-8') as f:
         f.write(contents)
     _git(directory, 'add', filename)
     _git(directory, 'commit', '-m', commit_message)
         f.write(contents)
     _git(directory, 'add', filename)
     _git(directory, 'commit', '-m', commit_message)