Skip to content

Conversation

@Pierre-Sassoulas
Copy link
Member

Type of Changes

Type
📜 Docs

Description

Refs #7897

There are multiple examples for consider-using-f-string and consider-using-generator: but I think it's not worth it to separate everything. (Better to keep all the examples together in this case).

@codecov
Copy link

codecov bot commented Sep 4, 2023

Codecov Report

Merging #9006 (6815f67) into main (1996149) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #9006   +/-   ##
=======================================
  Coverage   95.75%   95.75%           
=======================================
  Files         173      173           
  Lines       18645    18645           
=======================================
  Hits        17853    17853           
  Misses        792      792           

@@ -1,2 +1,2 @@
- `logging variable data <https://docs.python.org/3/howto/logging.html#logging-variable-data>`_
- `rational <https://stackoverflow.com/questions/34619790>`_
- `Rational for the message on stackoverflow <https://stackoverflow.com/a/34634301/2519059>`_
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- `Rational for the message on stackoverflow <https://stackoverflow.com/a/34634301/2519059>`_
- `Rationale for the message on stackoverflow <https://stackoverflow.com/a/34634301/2519059>`_

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked rational is the adjective so that's why spell correction did not work :D ! Thank you fixed in 42bac1f

@Pierre-Sassoulas Pierre-Sassoulas merged commit 9c1c663 into pylint-dev:main Sep 5, 2023
@Pierre-Sassoulas Pierre-Sassoulas deleted the use-new-multiple-file-template branch September 5, 2023 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation 📗 Skip news 🔇 This change does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants