Skip to content

Conversation

cdce8p
Copy link
Member

@cdce8p cdce8p commented Sep 16, 2025

Description

Add new checks for match class patterns.

  • invalid-match-args-definition
  • too-many-positional-sub-patterns
  • multiple-class-sub-patterns

@cdce8p cdce8p added this to the 4.0.0 milestone Sep 16, 2025
@cdce8p cdce8p added Enhancement ✨ Improvement to a component Match case labels Sep 16, 2025
@cdce8p cdce8p force-pushed the checks-match-class branch 2 times, most recently from 78a564c to 482e349 Compare September 16, 2025 13:31
Copy link

codecov bot commented Sep 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.90%. Comparing base (96f6b9c) to head (4936dc6).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #10559   +/-   ##
=======================================
  Coverage   95.89%   95.90%           
=======================================
  Files         176      176           
  Lines       19416    19453   +37     
=======================================
+ Hits        18619    18656   +37     
  Misses        797      797           
Files with missing lines Coverage Δ
pylint/checkers/match_statements_checker.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This comment has been minimized.

This comment has been minimized.

Copy link
Collaborator

@DanielNoord DanielNoord left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks :)

Copy link
Member

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

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

👌

@cdce8p cdce8p merged commit 59c000a into pylint-dev:main Sep 16, 2025
44 checks passed
@cdce8p cdce8p deleted the checks-match-class branch September 16, 2025 20:26
Copy link
Contributor

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

This comment was generated for commit 4936dc6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement ✨ Improvement to a component Match case
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants