Skip to content

Commit 36812a3

Browse files
authored
Merge pull request #377 from josegonzalez/fix-lint-errors
chore: reformat file to fix lint issues
2 parents eb545c1 + 0e01971 commit 36812a3

File tree

1 file changed

+40
-23
lines changed

1 file changed

+40
-23
lines changed

github_backup/github_backup.py

+40-23
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,12 @@
1919
import subprocess
2020
import sys
2121
import time
22+
from datetime import datetime
2223
from http.client import IncompleteRead
2324
from urllib.error import HTTPError, URLError
2425
from urllib.parse import quote as urlquote
2526
from urllib.parse import urlencode, urlparse
2627
from urllib.request import HTTPRedirectHandler, Request, build_opener, urlopen
27-
from datetime import datetime
2828

2929
try:
3030
from . import __version__
@@ -40,16 +40,21 @@
4040
https_ctx = ssl.create_default_context()
4141
if not https_ctx.get_ca_certs():
4242
import warnings
43-
warnings.warn('\n\nYOUR DEFAULT CA CERTS ARE EMPTY.\n' +
44-
'PLEASE POPULATE ANY OF:' +
45-
''.join([
46-
'\n - ' + x
47-
for x in ssl.get_default_verify_paths()
48-
if type(x) is str
49-
]) + '\n', stacklevel=2)
43+
44+
warnings.warn(
45+
"\n\nYOUR DEFAULT CA CERTS ARE EMPTY.\n"
46+
+ "PLEASE POPULATE ANY OF:"
47+
+ "".join(
48+
["\n - " + x for x in ssl.get_default_verify_paths() if type(x) is str]
49+
)
50+
+ "\n",
51+
stacklevel=2,
52+
)
5053
import certifi
54+
5155
https_ctx = ssl.create_default_context(cafile=certifi.where())
5256

57+
5358
def logging_subprocess(
5459
popenargs, stdout_log_level=logging.DEBUG, stderr_log_level=logging.ERROR, **kwargs
5560
):
@@ -524,7 +529,7 @@ def get_github_host(args):
524529

525530

526531
def read_file_contents(file_uri):
527-
return open(file_uri[len(FILE_URI_PREFIX):], "rt").readline().strip()
532+
return open(file_uri[len(FILE_URI_PREFIX) :], "rt").readline().strip()
528533

529534

530535
def get_github_repo_url(args, repository):
@@ -795,13 +800,15 @@ def download_file(url, path, auth, as_app=False, fine=False):
795800
if os.path.exists(path):
796801
return
797802

798-
request = _construct_request(per_page=100,
799-
page=1,
800-
query_args={},
801-
template=url,
802-
auth=auth,
803-
as_app=as_app,
804-
fine=fine)
803+
request = _construct_request(
804+
per_page=100,
805+
page=1,
806+
query_args={},
807+
template=url,
808+
auth=auth,
809+
as_app=as_app,
810+
fine=fine,
811+
)
805812
request.add_header("Accept", "application/octet-stream")
806813
opener = build_opener(S3HTTPRedirectHandler)
807814

@@ -944,11 +951,15 @@ def filter_repositories(args, unfiltered_repositories):
944951
if r.get("language") and r.get("language").lower() in languages
945952
] # noqa
946953
if name_regex:
947-
repositories = [r for r in repositories if "name" not in r or name_regex.match(r["name"])]
954+
repositories = [
955+
r for r in repositories if "name" not in r or name_regex.match(r["name"])
956+
]
948957
if args.skip_archived:
949958
repositories = [r for r in repositories if not r.get("archived")]
950959
if args.exclude:
951-
repositories = [r for r in repositories if "name" not in r or r["name"] not in args.exclude]
960+
repositories = [
961+
r for r in repositories if "name" not in r or r["name"] not in args.exclude
962+
]
952963

953964
return repositories
954965

@@ -1244,10 +1255,16 @@ def backup_releases(args, repo_cwd, repository, repos_template, include_assets=F
12441255
if args.skip_prerelease:
12451256
releases = [r for r in releases if not r["prerelease"] and not r["draft"]]
12461257

1247-
if args.number_of_latest_releases and args.number_of_latest_releases < len(releases):
1248-
releases.sort(key=lambda item: datetime.strptime(item["created_at"], "%Y-%m-%dT%H:%M:%SZ"),
1249-
reverse=True)
1250-
releases = releases[:args.number_of_latest_releases]
1258+
if args.number_of_latest_releases and args.number_of_latest_releases < len(
1259+
releases
1260+
):
1261+
releases.sort(
1262+
key=lambda item: datetime.strptime(
1263+
item["created_at"], "%Y-%m-%dT%H:%M:%SZ"
1264+
),
1265+
reverse=True,
1266+
)
1267+
releases = releases[: args.number_of_latest_releases]
12511268
logger.info("Saving the latest {0} releases to disk".format(len(releases)))
12521269
else:
12531270
logger.info("Saving {0} releases to disk".format(len(releases)))
@@ -1274,7 +1291,7 @@ def backup_releases(args, repo_cwd, repository, repos_template, include_assets=F
12741291
os.path.join(release_assets_cwd, asset["name"]),
12751292
get_auth(args, encode=not args.as_app),
12761293
as_app=args.as_app,
1277-
fine=True if args.token_fine is not None else False
1294+
fine=True if args.token_fine is not None else False,
12781295
)
12791296

12801297

0 commit comments

Comments
 (0)