Skip to content

[doc] Document invalid characters that exists because of the genericity of the unicode checker#9042

Merged
Pierre-Sassoulas merged 2 commits intopylint-dev:mainfrom
Pierre-Sassoulas:invalid-character-theoric-one
Sep 17, 2023
Merged

[doc] Document invalid characters that exists because of the genericity of the unicode checker#9042
Pierre-Sassoulas merged 2 commits intopylint-dev:mainfrom
Pierre-Sassoulas:invalid-character-theoric-one

Conversation

@Pierre-Sassoulas
Copy link
Copy Markdown
Member

Type of Changes

Type
📜 Docs

Description

Refs #5953

As far as I know it's impossible to raise those two messages unless you construct a theoretical node in a unit test.

@Pierre-Sassoulas Pierre-Sassoulas added Documentation 📗 Skip news 🔇 This change does not require a changelog entry labels Sep 17, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 17, 2023

Codecov Report

Merging #9042 (f85e422) into main (3f93f1e) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #9042   +/-   ##
=======================================
  Coverage   95.75%   95.75%           
=======================================
  Files         173      173           
  Lines       18652    18652           
=======================================
  Hits        17860    17860           
  Misses        792      792           

Comment thread doc/data/messages/i/invalid-character-carriage-return/details.rst Outdated
Refs pylint-dev#5953

Co-authored-by: Jacob Walls <jacobtylerwalls@gmail.com>
@Pierre-Sassoulas Pierre-Sassoulas force-pushed the invalid-character-theoric-one branch from 08efe18 to f85e422 Compare September 17, 2023 15:55
@Pierre-Sassoulas Pierre-Sassoulas merged commit 8691519 into pylint-dev:main Sep 17, 2023
@Pierre-Sassoulas Pierre-Sassoulas deleted the invalid-character-theoric-one branch September 17, 2023 16: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