Skip to content

Commit

Permalink
"Message emitted" improvements caused issues
Browse files Browse the repository at this point in the history
Wrapping "Message emitted" in monospaced formatting,
causes `W1507` (`pylint/checkers/stdlib.py`) to fail on a stray space.

Fix that - as it seems to be the only outlier.
(Might've been nice for a more formal check in the first place 🙏).

Signed-off-by: Stavros Ntentos <133706+stdedos@users.noreply.github.com>
  • Loading branch information
stdedos authored and Pierre-Sassoulas committed Feb 22, 2023
1 parent b995af4 commit f54dfd1
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion pylint/checkers/stdlib.py
Original file line number Diff line number Diff line change
Expand Up @@ -386,7 +386,7 @@ class StdlibChecker(DeprecatedMixin, BaseChecker):
"By default, the first parameter is the group param, not the target param.",
),
"W1507": (
"Using copy.copy(os.environ). Use os.environ.copy() instead. ",
"Using copy.copy(os.environ). Use os.environ.copy() instead.",
"shallow-copy-environ",
"os.environ is not a dict object but proxy object, so "
"shallow copy has still effects on original object. "
Expand Down
4 changes: 2 additions & 2 deletions tests/functional/s/shallow_copy_environ.txt
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
shallow-copy-environ:7:0:7:21::Using copy.copy(os.environ). Use os.environ.copy() instead. :UNDEFINED
shallow-copy-environ:17:0:17:18::Using copy.copy(os.environ). Use os.environ.copy() instead. :UNDEFINED
shallow-copy-environ:7:0:7:21::Using copy.copy(os.environ). Use os.environ.copy() instead.:UNDEFINED
shallow-copy-environ:17:0:17:18::Using copy.copy(os.environ). Use os.environ.copy() instead.:UNDEFINED

0 comments on commit f54dfd1

Please sign in to comment.