Skip to content

Conversation

cdce8p
Copy link
Member

@cdce8p cdce8p commented Sep 26, 2025

Some more cleanup.

@cdce8p cdce8p requested a review from DudeNr33 as a code owner September 26, 2025 20:38
@cdce8p cdce8p added Maintenance Discussion or action around maintaining pylint or the dev workflow Skip news 🔇 This change does not require a changelog entry labels Sep 26, 2025
Copy link

codecov bot commented Sep 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.95%. Comparing base (19e0f3e) to head (e459566).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #10594   +/-   ##
=======================================
  Coverage   95.95%   95.95%           
=======================================
  Files         176      176           
  Lines       19467    19476    +9     
=======================================
+ Hits        18680    18689    +9     
  Misses        787      787           
Files with missing lines Coverage Δ
pylint/checkers/async_checker.py 100.00% <100.00%> (ø)
pylint/checkers/base/basic_checker.py 98.32% <100.00%> (ø)
pylint/checkers/base/name_checker/checker.py 98.72% <100.00%> (-0.01%) ⬇️
pylint/checkers/classes/class_checker.py 94.01% <100.00%> (-0.01%) ⬇️
pylint/checkers/imports.py 94.88% <100.00%> (+0.01%) ⬆️
pylint/checkers/nested_min_max.py 100.00% <ø> (ø)
pylint/checkers/refactoring/refactoring_checker.py 98.22% <100.00%> (-0.01%) ⬇️
pylint/checkers/strings.py 94.29% <100.00%> (ø)
pylint/checkers/typecheck.py 96.50% <100.00%> (+0.01%) ⬆️
pylint/checkers/utils.py 95.97% <100.00%> (+<0.01%) ⬆️
... and 6 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

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

This comment was generated for commit e459566

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.

I don't care much and generally don't like module.Class references if Class is not defined in module or module is the preferred way to get it (e.g., module._private.Class`).

So LGTM!

@cdce8p cdce8p merged commit 0871a4d into pylint-dev:main Sep 28, 2025
48 checks passed
@cdce8p cdce8p deleted the update-astroid-imports branch September 28, 2025 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintenance Discussion or action around maintaining pylint or the dev workflow 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