Skip to content

[test] Add primer test cases for suboptimal diff scenarios#10946

Merged
Pierre-Sassoulas merged 1 commit intopylint-dev:mainfrom
Pierre-Sassoulas:add-primer-test-case
Mar 30, 2026
Merged

[test] Add primer test cases for suboptimal diff scenarios#10946
Pierre-Sassoulas merged 1 commit intopylint-dev:mainfrom
Pierre-Sassoulas:add-primer-test-case

Conversation

@Pierre-Sassoulas
Copy link
Copy Markdown
Member

Type of Changes

Type
📜 Docs

Description

Document current behavior where line-only moves, message text changes, type changes, and useless-suppression pairs all show as full remove+add in the primer comment. These cases will help validate future smarter diff logic in #10914

Document current behavior where line-only moves, message text changes,
type changes, and useless-suppression pairs all show as full remove+add
in the primer comment. These cases will help validate future smarter
diff logic.
@Pierre-Sassoulas Pierre-Sassoulas added Maintenance Discussion or action around maintaining pylint or the dev workflow Skip news 🔇 This change does not require a changelog entry labels Mar 30, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.17%. Comparing base (b93ffdb) to head (334ea75).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #10946   +/-   ##
=======================================
  Coverage   96.17%   96.17%           
=======================================
  Files         178      178           
  Lines       19634    19634           
=======================================
  Hits        18883    18883           
  Misses        751      751           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Pierre-Sassoulas Pierre-Sassoulas merged commit f12dfe1 into pylint-dev:main Mar 30, 2026
30 checks passed
@Pierre-Sassoulas Pierre-Sassoulas deleted the add-primer-test-case branch March 30, 2026 20:38
ranadheerg pushed a commit to ranadheerg/pylint that referenced this pull request Apr 10, 2026
…v#10946)

Document current behavior where line-only moves, message text changes,
type changes, and useless-suppression pairs all show as full remove+add
in the primer comment. These cases will help validate future smarter
diff logic.
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.

2 participants