Work on documentation for non-ascii-file-name

[ollie-iterators]

Type of Changes

	Type
✓	🔨 Refactoring
✓	📜 Docs

Description

Working on documentation for non-ascii-file-name

This is because while the issue (https://bugs.python.org/issue20485) that caused this issue has been fixed since Python 3.5. It is still not recommended to use non-ascii characters in a filename.

This is the PR that fixed the issue: python/cpython#64684 (comment).

Refs #7897

[codecov]

Codecov Report

Merging #8324 (9ee4f49) into main (8bf1120) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8324   +/-   ##
=======================================
  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% <ø> (ø)

[Pierre-Sassoulas]

This need a changelog message, and a cleanup of the existing code in the checkers, this is not going to be easy, a maintainer or someone experienced should probably do it. Also the current MR need to justify the removal if someone read it later.

[ollie-iterators]

Ok, I updated the MR

[Pierre-Sassoulas]

Thanks for adding the description and the changelog :) Next step would be to remove all the code related to this (functional tests, all the part of the pylint code specific to this check), do you want to do it or should we take over ?

[ollie-iterators]

Thanks for adding the description and the changelog :) Next step would be to remove all the code related to this (functional tests, all the part of the pylint code specific to this check), do you want to do it or should we take over ?

Thanks, but I think I'll try to do it

[ollie-iterators]

Ok, I removed non-ascii-file-name from all the files except for full.rst, summary.rst and the files that are necessary to show that it was removed.

[Pierre-Sassoulas]

All right thank you for reverting. Let's simply document the message here.

[github-actions]

🤖 According to the primer, this change has no effect on the checked open source code. 🤖🎉

This comment was generated for commit 9ee4f49

[Pierre-Sassoulas]

Thank you for working on this. Sorry I'm going to close because the first link is just something that point to PEP489 that we already linked, and the second link point to a list of non ascii character that is not relevant to the issue. Also we can actually document the problem properly see #8340