Skip to content

Commit 54da8b6

Browse files
Apply suggestions from code review
Co-authored-by: Ed Morley <[email protected]>
1 parent bd04fc6 commit 54da8b6

File tree

6 files changed

+10
-10
lines changed

6 files changed

+10
-10
lines changed

setuptools/_normalization.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ def best_effort_version(version: str) -> str:
8181
f"""
8282
Version {version!r} is not valid according to PEP 440.
8383
84-
Please make sure specify a valid version for your package.
84+
Please make sure to specify a valid version for your package.
8585
Also note that future releases of setuptools may halt the build process
8686
if an invalid version is given.
8787
""",

setuptools/build_meta.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -304,7 +304,7 @@ def _arbitrary_args(self, config_settings: _ConfigSettings) -> Iterator[str]:
304304
f"""
305305
The arguments {bad_args!r} were given via `--global-option`.
306306
Please use `--build-option` instead,
307-
`--global-option` is reserved to flags like `--verbose` or `--quiet`.
307+
`--global-option` is reserved for flags like `--verbose` or `--quiet`.
308308
""",
309309
due_date=(2023, 9, 26), # Warning introduced in v64.0.1, 11/Aug/2022.
310310
)

setuptools/command/build.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ def get_sub_commands(self):
2323
subcommands = {cmd[0] for cmd in _build.sub_commands}
2424
if subcommands - _ORIGINAL_SUBCOMMANDS:
2525
SetuptoolsDeprecationWarning.emit(
26-
"Directly usage of `distutils` commands",
26+
"Direct usage of `distutils` commands",
2727
"""
2828
It seems that you are using `distutils.command.build` to add
2929
new subcommands. Using `distutils` directly is considered deprecated,

setuptools/command/build_py.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -336,9 +336,9 @@ class _Warning(SetuptoolsDeprecationWarning):
336336
Python recognizes {importable!r} as an importable package[^1],
337337
but it is absent from setuptools' `packages` configuration.
338338
339-
This leads to an ambiguous overall configuration. If you want distribute this
339+
This leads to an ambiguous overall configuration. If you want to distribute this
340340
package, please make sure that {importable!r} is explicitly added
341-
the `packages` configuration field.
341+
to the `packages` configuration field.
342342
343343
Alternatively, you can also rely on setuptools' discovery methods
344344
(for example by using `find_namespace_packages(...)`/`find_namespace:`

setuptools/command/editable_wheel.py

+4-4
Original file line numberDiff line numberDiff line change
@@ -305,7 +305,7 @@ def _safely_run(self, cmd_name: str):
305305
If you are seeing this warning it is very likely that a setuptools
306306
plugin or customization overrides the `{cmd_name}` command, without
307307
taking into consideration how editable installs run build steps
308-
starting from v64.0.0.
308+
starting from setuptools v64.0.0.
309309
310310
Plugin authors and developers relying on custom build steps are
311311
encouraged to update their `{cmd_name}` implementation considering the
@@ -836,18 +836,18 @@ class _DebuggingTips(SetuptoolsWarning):
836836
_DETAILS = """
837837
An error happened while installing `{project}` in editable mode.
838838
839-
The following steps are recommended to help debugging this problem:
839+
The following steps are recommended to help debug this problem:
840840
841841
- Try to install the project normally, without using the editable mode.
842-
Does the error still persists?
842+
Does the error still persist?
843843
(If it does, try fixing the problem before attempting the editable mode).
844844
- If you are using binary extensions, make sure you have all OS-level
845845
dependencies installed (e.g. compilers, toolchains, binary libraries, ...).
846846
- Try the latest version of setuptools (maybe the error was already fixed).
847847
- If you (or your project dependencies) are using any setuptools extension
848848
or customization, make sure they support the editable mode.
849849
850-
After following the steps above, if the problem still persist and
850+
After following the steps above, if the problem still persists and
851851
you think this is related to how setuptools handles editable installations,
852852
please submit a reproducible example
853853
(see https://stackoverflow.com/help/minimal-reproducible-example) to:

setuptools/dist.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -825,7 +825,7 @@ def make_option_lowercase(self, opt, section):
825825
"Invalid uppercase configuration",
826826
f"""
827827
Usage of uppercase key {opt!r} in {section!r} will not be supported in
828-
future versions. Please use lowercase {lowercase_opt!r} instead
828+
future versions. Please use lowercase {lowercase_opt!r} instead.
829829
""",
830830
see_docs="userguide/declarative_config.html",
831831
due_date=(2023, 9, 26),

0 commit comments

Comments
 (0)