Skip to content

Conversation

@Pierre-Sassoulas
Copy link
Member

Type of Changes

Type
βœ“ πŸ“œ Docs

Description

Move links to related.rst, and upgrade the message shown for python 3.5+

Replace #8324, refs #5953

And move links to related.rst
@Pierre-Sassoulas Pierre-Sassoulas force-pushed the document-non-ascii-file-name branch from ad657d2 to 7d3c34c Compare February 25, 2023 12:33
@codecov
Copy link

codecov bot commented Feb 25, 2023

Codecov Report

Merging #8340 (30258f2) into main (8bf1120) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #8340   +/-   ##
=======================================
  Coverage   95.46%   95.46%           
=======================================
  Files         177      177           
  Lines       18704    18704           
=======================================
  Hits        17856    17856           
  Misses        848      848           
Impacted Files Coverage Ξ”
pylint/checkers/non_ascii_names.py 100.00% <ΓΈ> (ΓΈ)

@github-actions

This comment has been minimized.

@ollie-iterators
Copy link
Contributor

ollie-iterators commented Feb 25, 2023

This was a tricky issue to work on. Good job on the pull request and I like that you used the file name "not_better", very funny.

**Default:** ``True``


--source-roots
Copy link
Member

Choose a reason for hiding this comment

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

Is this meant to be part of this MR?

Copy link
Member Author

Choose a reason for hiding this comment

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

We forgot to generate the doc previousely because of #6736

@mbyrnepr2
Copy link
Member

I'm not sure the rationale behind the (empty) good/bad.py file or the good/not_better.py files?

Copy link
Member

@mbyrnepr2 mbyrnepr2 left a comment

Choose a reason for hiding this comment

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

Just a few small comments :)

@Pierre-Sassoulas
Copy link
Member Author

Pierre-Sassoulas commented Feb 25, 2023

I'm not sure the rationale behind the (empty) good/bad.py file or the good/not_better.py files?

The problem is in the name of the file so we don't need any content imo. Maybe something like """'Γ ' is not an ascii character but the file is named 'bΓ d.py'"""' ?

@mbyrnepr2
Copy link
Member

I'm not sure the rationale behind the (empty) good/bad.py file or the good/not_better.py files?

The problem is in the name of the file so we don't need any content imo. Maybe something like """'Γ ' is not an ascii character but the file is named 'bΓ d.py'"""' ?

I meant the file paths themselves. What does good/bad.py mean? πŸ˜„

@Pierre-Sassoulas
Copy link
Member Author

I meant the file paths themselves. What does good/bad.py mean?

Ha, the doc framework render it as "correct code" vs "bad code" automatically see https://pylint--8340.org.readthedocs.build/en/8340/user_guide/messages/warning/non-ascii-file-name.html. There's two files because we assumed that a directory could be used only if there was multiple file to demonstrate and if there's only one file it need to be named bad.py.

@github-actions
Copy link
Contributor

πŸ€– According to the primer, this change has no effect on the checked open source code. πŸ€–πŸŽ‰

This comment was generated for commit 30258f2

@mbyrnepr2
Copy link
Member

I meant the file paths themselves. What does good/bad.py mean?

Ha, the doc framework render it as "correct code" vs "bad code" automatically see https://pylint--8340.org.readthedocs.build/en/8340/user_guide/messages/warning/non-ascii-file-name.html. There's two files because we assumed that a directory could be used only if there was multiple file to demonstrate and if there's only one file it need to be named bad.py.

Awesome 😎

@Pierre-Sassoulas Pierre-Sassoulas merged commit c785e3e into pylint-dev:main Feb 25, 2023
@Pierre-Sassoulas Pierre-Sassoulas deleted the document-non-ascii-file-name branch February 25, 2023 20:39
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.

3 participants