Skip to content

Commit 988c436

Browse files
dosasdosas
committed
Add changes from black command
black .
1 parent 3ec7d4a commit 988c436

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

44 files changed

+827
-579
lines changed

‎.ci/scripts/calc_constraints.py‎

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,7 @@
1919
import tomli as tomllib
2020

2121

22-
CORE_TEMPLATE_URL = (
23-
"https://raw.githubusercontent.com/pulp/pulpcore/main/template_config.yml"
24-
)
22+
CORE_TEMPLATE_URL = "https://raw.githubusercontent.com/pulp/pulpcore/main/template_config.yml"
2523

2624

2725
def fetch_pulpcore_upper_bound(requirement):
@@ -94,9 +92,7 @@ def to_lower_bound(req):
9492
# TODO Semver libraries should be allowed too.
9593
operator = "~="
9694
if len(Version(min_version).release) != 3:
97-
raise RuntimeError(
98-
"Pulpcore lower bound must be in the form '>=x.y.z'."
99-
)
95+
raise RuntimeError("Pulpcore lower bound must be in the form '>=x.y.z'.")
10096
else:
10197
operator = "=="
10298
return f"{requirement.name}{operator}{min_version}"
@@ -116,14 +112,8 @@ def main():
116112

117113
modifier = to_upper_bound if args.upper else to_lower_bound
118114

119-
req_files = [
120-
filename
121-
for filename in args.filename
122-
if not filename.endswith("pyproject.toml")
123-
]
124-
pyp_files = [
125-
filename for filename in args.filename if filename.endswith("pyproject.toml")
126-
]
115+
req_files = [filename for filename in args.filename if not filename.endswith("pyproject.toml")]
116+
pyp_files = [filename for filename in args.filename if filename.endswith("pyproject.toml")]
127117
if req_files:
128118
with fileinput.input(files=req_files) as req_file:
129119
for line in req_file:

‎.ci/scripts/check_requirements.py‎

Lines changed: 5 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -53,14 +53,10 @@ def main():
5353
if line.startswith("git+"):
5454
# The single exception...
5555
if "pulp-smash" not in line:
56-
errors.append(
57-
f"{filename}:{nr}: Invalid source requirement: {line}"
58-
)
56+
errors.append(f"{filename}:{nr}: Invalid source requirement: {line}")
5957
elif line.startswith("-r "):
6058
if check_r:
61-
errors.append(
62-
f"{filename}:{nr}: Invalid deferred requirement: {line}"
63-
)
59+
errors.append(f"{filename}:{nr}: Invalid deferred requirement: {line}")
6460
else:
6561
errors.append(f"{filename}:{nr}: Unreadable requirement {line}")
6662
else:
@@ -70,19 +66,13 @@ def main():
7066
not req.name.startswith("opentelemetry")
7167
and req.name != "pulp-python-client"
7268
):
73-
errors.append(
74-
f"{filename}:{nr}: Prerelease versions found in {line}."
75-
)
69+
errors.append(f"{filename}:{nr}: Prerelease versions found in {line}.")
7670
ops = [spec.operator for spec in req.specifier]
7771
if "~=" in ops:
78-
warnings.warn(
79-
f"{filename}:{nr}: Please avoid using ~= on {req.name}!"
80-
)
72+
warnings.warn(f"{filename}:{nr}: Please avoid using ~= on {req.name}!")
8173
elif "<" not in ops and "<=" not in ops and "==" not in ops:
8274
if check_upperbound:
83-
errors.append(
84-
f"{filename}:{nr}: Upper bound missing in {line}."
85-
)
75+
errors.append(f"{filename}:{nr}: Upper bound missing in {line}.")
8676
except FileNotFoundError:
8777
# skip this test for plugins that don't use this requirements.txt
8878
pass

‎.ci/scripts/collect_changes.py‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -45,9 +45,7 @@
4545
TITLE_REGEX = (
4646
"("
4747
+ re.escape(
48-
TITLE_FORMAT.format(
49-
name="NAME_REGEX", version="VERSION_REGEX", project_date="DATE_REGEX"
50-
)
48+
TITLE_FORMAT.format(name="NAME_REGEX", version="VERSION_REGEX", project_date="DATE_REGEX")
5149
)
5250
.replace("NAME_REGEX", NAME_REGEX)
5351
.replace("VERSION_REGEX", VERSION_CAPTURE_REGEX, 1)
@@ -72,17 +70,13 @@ def _tokenize_changes(splits):
7270
def split_changelog(changelog):
7371
preamble, rest = changelog.split(START_STRING, maxsplit=1)
7472
split_rest = re.split(TITLE_REGEX, rest)
75-
return preamble + START_STRING + split_rest[0], list(
76-
_tokenize_changes(split_rest[1:])
77-
)
73+
return preamble + START_STRING + split_rest[0], list(_tokenize_changes(split_rest[1:]))
7874

7975

8076
def main():
8177
repo = Repo(os.getcwd())
8278
remote = repo.remotes[0]
83-
branches = [
84-
ref for ref in remote.refs if re.match(r"^([0-9]+)\.([0-9]+)$", ref.remote_head)
85-
]
79+
branches = [ref for ref in remote.refs if re.match(r"^([0-9]+)\.([0-9]+)$", ref.remote_head)]
8680
branches.sort(key=lambda ref: parse_version(ref.remote_head), reverse=True)
8781
branches = [ref.name for ref in branches]
8882

‎.ci/scripts/schema.py‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
But some pulp paths start with curly brackets e.g. {artifact_href}
88
This script modifies drf-spectacular schema validation to accept slashes and curly brackets.
99
"""
10+
1011
import json
1112
from drf_spectacular.validation import JSON_SCHEMA_SPEC_PATH
1213

‎.github/workflows/scripts/stage-changelog-for-default-branch.py‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,7 @@
2323
"""
2424
)
2525

26-
parser = argparse.ArgumentParser(
27-
formatter_class=argparse.RawTextHelpFormatter, description=helper
28-
)
26+
parser = argparse.ArgumentParser(formatter_class=argparse.RawTextHelpFormatter, description=helper)
2927

3028
parser.add_argument(
3129
"release_version",

‎.github/workflows/scripts/update_backport_labels.py‎

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,9 @@ def random_color():
2828
session.headers.update(headers)
2929

3030
# get all labels from the repository's current state
31-
response = session.get(
32-
"https://api.github.com/repos/pulp/pulp_python/labels", headers=headers
33-
)
31+
response = session.get("https://api.github.com/repos/pulp/pulp_python/labels", headers=headers)
3432
assert response.status_code == 200
35-
old_labels = set(
36-
[x["name"] for x in response.json() if x["name"].startswith("backport-")]
37-
)
33+
old_labels = set([x["name"] for x in response.json() if x["name"].startswith("backport-")])
3834

3935
# get list of branches from template_config.yml
4036
with open("./template_config.yml", "r") as f:

‎pulp_python/__init__.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
default_app_config = 'pulp_python.app.PulpPythonPluginAppConfig'
1+
default_app_config = "pulp_python.app.PulpPythonPluginAppConfig"

‎pulp_python/app/management/commands/repair-python-metadata.py‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ def repair_metadata(content):
2323
batch = []
2424
set_of_update_fields = set()
2525
total_repaired = 0
26-
for package in immediate_content.prefetch_related('_artifacts').iterator(chunk_size=1000):
26+
for package in immediate_content.prefetch_related("_artifacts").iterator(chunk_size=1000):
2727
new_data = artifact_to_python_content_data(
2828
package.filename, package._artifacts.get(), package.pulp_domain
2929
)
@@ -55,7 +55,7 @@ def href_prn_list_handler(value):
5555
(?:{settings.API_ROOT}(?:[-_a-zA-Z0-9]+/)?api/v3/repositories/python/python/[-a-f0-9]+/)
5656
|(?:prn:python\.pythonrepository:[-a-f0-9]+)
5757
""",
58-
re.VERBOSE
58+
re.VERBOSE,
5959
)
6060
values = []
6161
for v in value.split(","):

‎pulp_python/app/migrations/0001_initial.py‎

Lines changed: 127 additions & 50 deletions
Original file line numberDiff line numberDiff line change
@@ -10,87 +10,164 @@ class Migration(migrations.Migration):
1010
initial = True
1111

1212
dependencies = [
13-
('core', '0091_systemid'),
13+
("core", "0091_systemid"),
1414
]
1515

1616
operations = [
1717
migrations.CreateModel(
18-
name='PythonPublication',
18+
name="PythonPublication",
1919
fields=[
20-
('publication_ptr', models.OneToOneField(auto_created=True, on_delete=django.db.models.deletion.CASCADE, parent_link=True, primary_key=True, related_name='python_pythonpublication', serialize=False, to='core.publication')),
20+
(
21+
"publication_ptr",
22+
models.OneToOneField(
23+
auto_created=True,
24+
on_delete=django.db.models.deletion.CASCADE,
25+
parent_link=True,
26+
primary_key=True,
27+
related_name="python_pythonpublication",
28+
serialize=False,
29+
to="core.publication",
30+
),
31+
),
2132
],
2233
options={
23-
'default_related_name': '%(app_label)s_%(model_name)s',
34+
"default_related_name": "%(app_label)s_%(model_name)s",
2435
},
25-
bases=('core.publication',),
36+
bases=("core.publication",),
2637
),
2738
migrations.CreateModel(
28-
name='PythonRemote',
39+
name="PythonRemote",
2940
fields=[
30-
('remote_ptr', models.OneToOneField(auto_created=True, on_delete=django.db.models.deletion.CASCADE, parent_link=True, primary_key=True, related_name='python_pythonremote', serialize=False, to='core.remote')),
31-
('prereleases', models.BooleanField(default=False)),
32-
('includes', django.contrib.postgres.fields.jsonb.JSONField(default=list)),
33-
('excludes', django.contrib.postgres.fields.jsonb.JSONField(default=list)),
41+
(
42+
"remote_ptr",
43+
models.OneToOneField(
44+
auto_created=True,
45+
on_delete=django.db.models.deletion.CASCADE,
46+
parent_link=True,
47+
primary_key=True,
48+
related_name="python_pythonremote",
49+
serialize=False,
50+
to="core.remote",
51+
),
52+
),
53+
("prereleases", models.BooleanField(default=False)),
54+
("includes", django.contrib.postgres.fields.jsonb.JSONField(default=list)),
55+
("excludes", django.contrib.postgres.fields.jsonb.JSONField(default=list)),
3456
],
3557
options={
36-
'default_related_name': '%(app_label)s_%(model_name)s',
58+
"default_related_name": "%(app_label)s_%(model_name)s",
3759
},
38-
bases=('core.remote',),
60+
bases=("core.remote",),
3961
),
4062
migrations.CreateModel(
41-
name='PythonRepository',
63+
name="PythonRepository",
4264
fields=[
43-
('repository_ptr', models.OneToOneField(auto_created=True, on_delete=django.db.models.deletion.CASCADE, parent_link=True, primary_key=True, related_name='python_pythonrepository', serialize=False, to='core.repository')),
65+
(
66+
"repository_ptr",
67+
models.OneToOneField(
68+
auto_created=True,
69+
on_delete=django.db.models.deletion.CASCADE,
70+
parent_link=True,
71+
primary_key=True,
72+
related_name="python_pythonrepository",
73+
serialize=False,
74+
to="core.repository",
75+
),
76+
),
4477
],
4578
options={
46-
'default_related_name': '%(app_label)s_%(model_name)s',
79+
"default_related_name": "%(app_label)s_%(model_name)s",
4780
},
48-
bases=('core.repository',),
81+
bases=("core.repository",),
4982
),
5083
migrations.CreateModel(
51-
name='PythonPackageContent',
84+
name="PythonPackageContent",
5285
fields=[
53-
('content_ptr', models.OneToOneField(auto_created=True, on_delete=django.db.models.deletion.CASCADE, parent_link=True, primary_key=True, related_name='python_pythonpackagecontent', serialize=False, to='core.content')),
54-
('filename', models.TextField(db_index=True, unique=True)),
55-
('packagetype', models.TextField(choices=[('bdist_dmg', 'bdist_dmg'), ('bdist_dumb', 'bdist_dumb'), ('bdist_egg', 'bdist_egg'), ('bdist_msi', 'bdist_msi'), ('bdist_rpm', 'bdist_rpm'), ('bdist_wheel', 'bdist_wheel'), ('bdist_wininst', 'bdist_wininst'), ('sdist', 'sdist')])),
56-
('name', models.TextField()),
57-
('version', models.TextField()),
58-
('metadata_version', models.TextField()),
59-
('summary', models.TextField()),
60-
('description', models.TextField()),
61-
('keywords', models.TextField()),
62-
('home_page', models.TextField()),
63-
('download_url', models.TextField()),
64-
('author', models.TextField()),
65-
('author_email', models.TextField()),
66-
('maintainer', models.TextField()),
67-
('maintainer_email', models.TextField()),
68-
('license', models.TextField()),
69-
('requires_python', models.TextField()),
70-
('project_url', models.TextField()),
71-
('platform', models.TextField()),
72-
('supported_platform', models.TextField()),
73-
('requires_dist', django.contrib.postgres.fields.jsonb.JSONField(default=list)),
74-
('provides_dist', django.contrib.postgres.fields.jsonb.JSONField(default=list)),
75-
('obsoletes_dist', django.contrib.postgres.fields.jsonb.JSONField(default=list)),
76-
('requires_external', django.contrib.postgres.fields.jsonb.JSONField(default=list)),
77-
('classifiers', django.contrib.postgres.fields.jsonb.JSONField(default=list)),
86+
(
87+
"content_ptr",
88+
models.OneToOneField(
89+
auto_created=True,
90+
on_delete=django.db.models.deletion.CASCADE,
91+
parent_link=True,
92+
primary_key=True,
93+
related_name="python_pythonpackagecontent",
94+
serialize=False,
95+
to="core.content",
96+
),
97+
),
98+
("filename", models.TextField(db_index=True, unique=True)),
99+
(
100+
"packagetype",
101+
models.TextField(
102+
choices=[
103+
("bdist_dmg", "bdist_dmg"),
104+
("bdist_dumb", "bdist_dumb"),
105+
("bdist_egg", "bdist_egg"),
106+
("bdist_msi", "bdist_msi"),
107+
("bdist_rpm", "bdist_rpm"),
108+
("bdist_wheel", "bdist_wheel"),
109+
("bdist_wininst", "bdist_wininst"),
110+
("sdist", "sdist"),
111+
]
112+
),
113+
),
114+
("name", models.TextField()),
115+
("version", models.TextField()),
116+
("metadata_version", models.TextField()),
117+
("summary", models.TextField()),
118+
("description", models.TextField()),
119+
("keywords", models.TextField()),
120+
("home_page", models.TextField()),
121+
("download_url", models.TextField()),
122+
("author", models.TextField()),
123+
("author_email", models.TextField()),
124+
("maintainer", models.TextField()),
125+
("maintainer_email", models.TextField()),
126+
("license", models.TextField()),
127+
("requires_python", models.TextField()),
128+
("project_url", models.TextField()),
129+
("platform", models.TextField()),
130+
("supported_platform", models.TextField()),
131+
("requires_dist", django.contrib.postgres.fields.jsonb.JSONField(default=list)),
132+
("provides_dist", django.contrib.postgres.fields.jsonb.JSONField(default=list)),
133+
("obsoletes_dist", django.contrib.postgres.fields.jsonb.JSONField(default=list)),
134+
("requires_external", django.contrib.postgres.fields.jsonb.JSONField(default=list)),
135+
("classifiers", django.contrib.postgres.fields.jsonb.JSONField(default=list)),
78136
],
79137
options={
80-
'default_related_name': '%(app_label)s_%(model_name)s',
81-
'unique_together': {('filename',)},
138+
"default_related_name": "%(app_label)s_%(model_name)s",
139+
"unique_together": {("filename",)},
82140
},
83-
bases=('core.content',),
141+
bases=("core.content",),
84142
),
85143
migrations.CreateModel(
86-
name='PythonDistribution',
144+
name="PythonDistribution",
87145
fields=[
88-
('basedistribution_ptr', models.OneToOneField(auto_created=True, on_delete=django.db.models.deletion.CASCADE, parent_link=True, primary_key=True, related_name='python_pythondistribution', serialize=False, to='core.basedistribution')),
89-
('publication', models.ForeignKey(null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='python_pythondistribution', to='core.publication')),
146+
(
147+
"basedistribution_ptr",
148+
models.OneToOneField(
149+
auto_created=True,
150+
on_delete=django.db.models.deletion.CASCADE,
151+
parent_link=True,
152+
primary_key=True,
153+
related_name="python_pythondistribution",
154+
serialize=False,
155+
to="core.basedistribution",
156+
),
157+
),
158+
(
159+
"publication",
160+
models.ForeignKey(
161+
null=True,
162+
on_delete=django.db.models.deletion.SET_NULL,
163+
related_name="python_pythondistribution",
164+
to="core.publication",
165+
),
166+
),
90167
],
91168
options={
92-
'default_related_name': '%(app_label)s_%(model_name)s',
169+
"default_related_name": "%(app_label)s_%(model_name)s",
93170
},
94-
bases=('core.basedistribution',),
171+
bases=("core.basedistribution",),
95172
),
96173
]

‎pulp_python/app/migrations/0002_pythonpackagecontent_python_version.py‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,14 @@
66
class Migration(migrations.Migration):
77

88
dependencies = [
9-
('python', '0001_initial'),
9+
("python", "0001_initial"),
1010
]
1111

1212
operations = [
1313
migrations.AddField(
14-
model_name='pythonpackagecontent',
15-
name='python_version',
16-
field=models.TextField(default=''),
14+
model_name="pythonpackagecontent",
15+
name="python_version",
16+
field=models.TextField(default=""),
1717
preserve_default=False,
1818
),
1919
]

0 commit comments

Comments
 (0)