pylint icon indicating copy to clipboard operation
pylint copied to clipboard

Fix: [unknown-option-value] False negative after --disable=all

Open fellhorn opened this issue 1 year ago • 5 comments

Type of Changes

Type
:bug: Bug

Description

Closes #9403 by excluding a set of internal pylint messages from --disable=all.

As this is my first PR to pylint I appreciate any comment on the solution. Especially in regards of the message codes and having to change an existing test.

fellhorn avatar Mar 09 '24 10:03 fellhorn

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

This comment was generated for commit 302e5135fb9d0a1f556422a0273a99010c5e1a74

github-actions[bot] avatar Mar 09 '24 10:03 github-actions[bot]

This PR needs take over because because it has been open 8 weeks with no activity.

github-actions[bot] avatar May 14 '24 02:05 github-actions[bot]

This PR needs take over because because it has been open 8 weeks with no activity.

I plan to take another look but lately had not really much capacity. If the issue is urgent though, please feel free to take over.

fellhorn avatar May 14 '24 07:05 fellhorn

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

This comment was generated for commit 350927040858e8755d4d8ef74b70cca7b56adf10

github-actions[bot] avatar Jun 02 '24 15:06 github-actions[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.79%. Comparing base (6888fde) to head (6548d2c). Report is 107 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #9491      +/-   ##
==========================================
- Coverage   95.82%   95.79%   -0.03%     
==========================================
  Files         174      174              
  Lines       18813    18925     +112     
==========================================
+ Hits        18027    18130     +103     
- Misses        786      795       +9     
Files Coverage Δ
pylint/lint/message_state_handler.py 96.53% <100.00%> (+0.05%) :arrow_up:
pylint/lint/run.py 87.59% <100.00%> (+0.09%) :arrow_up:

... and 31 files with indirect coverage changes

codecov[bot] avatar Jun 16 '24 08:06 codecov[bot]

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

This comment was generated for commit 453c9e10a75975d13cc9d029c43e424f4a34c503

github-actions[bot] avatar Jul 14 '24 19:07 github-actions[bot]

You can pip install -r doc/requirements.txt and then towncrier create 9403.bugfix to create a changelog stub.

jacobtylerwalls avatar Jul 21 '24 13:07 jacobtylerwalls

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

This comment was generated for commit 6548d2c4cb3dd8e5296ac432bfecbcd532a44508

github-actions[bot] avatar Jul 29 '24 10:07 github-actions[bot]