Skip to content

Commit

Permalink
Manual fixes after ruff check --fix runs
Browse files Browse the repository at this point in the history
  • Loading branch information
DimitriPapadopoulos committed Aug 18, 2024
1 parent 7801e94 commit 661f48e
Show file tree
Hide file tree
Showing 7 changed files with 24 additions and 19 deletions.
11 changes: 7 additions & 4 deletions pkg_resources/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,9 @@ def get_supported_platform():
m = macosVersionString.match(plat)
if m is not None and sys.platform == "darwin":
try:
plat = 'macosx-{}-{}'.format('.'.join(_macos_vers()[:2]), m.group(3))
major_minor = '.'.join(_macos_vers()[:2])
build = m.group(3)
plat = f'macosx-{major_minor}-{build}'
except ValueError:
# not macOS
pass
Expand Down Expand Up @@ -2723,7 +2725,8 @@ def __str__(self):
if self.attrs:
s += ':' + '.'.join(self.attrs)
if self.extras:
s += ' [{}]'.format(','.join(self.extras))
extras = ','.join(self.extras)
s += f' [{extras}]'
return s

def __repr__(self):
Expand Down Expand Up @@ -3309,8 +3312,8 @@ def check_version_conflict(self):
):
continue
issue_warning(
f"Module {modname} was already imported from {fn}, but {self.location} is being added"
" to sys.path",
f"Module {modname} was already imported from {fn}, "
f"but {self.location} is being added to sys.path",
)

def has_version(self):
Expand Down
4 changes: 2 additions & 2 deletions setuptools/_core_metadata.py
Original file line number Diff line number Diff line change
Expand Up @@ -178,8 +178,8 @@ def write_field(key, value):
if license:
write_field('License', rfc822_escape(license))

for project_url in self.project_urls.items():
write_field('Project-URL', '{}, {}'.format(*project_url))
for label, url in self.project_urls.items():
write_field('Project-URL', f'{label}, {url}')

keywords = ','.join(self.get_keywords())
if keywords:
Expand Down
2 changes: 1 addition & 1 deletion setuptools/command/bdist_egg.py
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ def __bootstrap__():


class bdist_egg(Command):
description = "create an \"egg\" distribution"
description = 'create an "egg" distribution'

user_options = [
('bdist-dir=', 'b', "temporary directory for creating the distribution"),
Expand Down
7 changes: 4 additions & 3 deletions setuptools/command/easy_install.py
Original file line number Diff line number Diff line change
Expand Up @@ -131,8 +131,8 @@ class easy_install(Command):
(
'optimize=',
'O',
"also compile with optimization: -O1 for \"python -O\", "
"-O2 for \"python -OO\", and -O0 to disable [default: -O0]",
'also compile with optimization: -O1 for "python -O", '
'-O2 for "python -OO", and -O0 to disable [default: -O0]',
),
('record=', None, "filename in which to record list of installed files"),
('always-unzip', 'Z', "don't install as a zipfile, no matter what"),
Expand Down Expand Up @@ -1021,7 +1021,8 @@ def install_exe(self, dist_filename, tmpdir):
f.write('Metadata-Version: 1.0\n')
for k, v in cfg.items('metadata'):
if k != 'target_version':
f.write('{}: {}\n'.format(k.replace('_', '-').title(), v))
k = k.replace('_', '-').title()
f.write(f'{k}: {v}\n')
script_dir = os.path.join(_egg_info, 'scripts')
# delete entry-point scripts to avoid duping
self.delete_blockers([
Expand Down
8 changes: 4 additions & 4 deletions setuptools/tests/config/test_setupcfg.py
Original file line number Diff line number Diff line change
Expand Up @@ -87,24 +87,24 @@ def test_basic(self, tmpdir):
'[options]\n'
'scripts = bin/a.py, bin/b.py\n',
)
config_dict = read_configuration(f'{config}')
config_dict = read_configuration(str(config))
assert config_dict['metadata']['version'] == '10.1.1'
assert config_dict['metadata']['keywords'] == ['one', 'two']
assert config_dict['options']['scripts'] == ['bin/a.py', 'bin/b.py']

def test_no_config(self, tmpdir):
with pytest.raises(DistutilsFileError):
read_configuration('{}'.format(tmpdir.join('setup.cfg')))
read_configuration(str(tmpdir.join('setup.cfg')))

def test_ignore_errors(self, tmpdir):
_, config = fake_env(
tmpdir,
'[metadata]\nversion = attr: none.VERSION\nkeywords = one, two\n',
)
with pytest.raises(ImportError):
read_configuration(f'{config}')
read_configuration(str(config))

config_dict = read_configuration(f'{config}', ignore_option_errors=True)
config_dict = read_configuration(str(config), ignore_option_errors=True)

assert config_dict['metadata']['keywords'] == ['one', 'two']
assert 'version' not in config_dict['metadata']
Expand Down
4 changes: 2 additions & 2 deletions setuptools/tests/test_build_ext.py
Original file line number Diff line number Diff line change
Expand Up @@ -285,8 +285,8 @@ def test_build_ext_config_handling(tmpdir_cwd):
),
}
path.build(files)
code, output = environment.run_setup_py(
code, (stdout, stderr) = environment.run_setup_py(
cmd=['build'],
data_stream=(0, 2),
)
assert code == 0, '\nSTDOUT:\n{}\nSTDERR:\n{}'.format(*output)
assert code == 0, f'\nSTDOUT:\n{stdout}\nSTDERR:\n{stderr}'
7 changes: 4 additions & 3 deletions setuptools/tests/test_egg_info.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,13 +33,14 @@ def env():
subs = 'home', 'lib', 'scripts', 'data', 'egg-base'
env.paths = dict((dirname, os.path.join(env_dir, dirname)) for dirname in subs)
list(map(os.mkdir, env.paths.values()))
egg_base = env.paths["egg-base"]
path.build({
env.paths['home']: {
'.pydistutils.cfg': DALS(
"""
f"""
[egg_info]
egg-base = {egg-base}
""".format(**env.paths)
egg-base = {egg_base}
"""
)
}
})
Expand Down

0 comments on commit 661f48e

Please sign in to comment.