Skip to content

Fix Ruff lint violations - #28

Merged
timkpaine merged 1 commit into
mainfrom
fix-ruff-lint
Jul 28, 2026
Merged

Fix Ruff lint violations#28
timkpaine merged 1 commit into
mainfrom
fix-ruff-lint

Conversation

@timkpaine

Copy link
Copy Markdown
Member

Description

Fix violations exposed by recent Ruff changes without modifying lint rules. Remove Python 3.10 compatibility code now that Python 3.11 is minimum, make subprocess behavior explicit, and apply Ruff-required simplifications and annotations.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor / code cleanup
  • CI / build configuration
  • Other (describe below)

Checklist

  • Linting passes (make lint)
  • Tests pass (make test)
  • New tests added for new functionality (not applicable; no new functionality)
  • Documentation updated (not applicable; no user-facing change)
  • Changelog / version bump (not applicable; no release requested)

Signed-off-by: Tim Paine <3105306+timkpaine@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

Test Results

133 tests  ±0   133 ✅ ±0   7s ⏱️ -1s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 2be96b4. ± Comparison against base commit b8ab1a9.

@codecov

codecov Bot commented Jul 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 88.23529% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.68%. Comparing base (b8ab1a9) to head (2be96b4).

Files with missing lines Patch % Lines
check_dist/_core.py 80.00% 1 Missing ⚠️
check_dist/tests/test_all.py 90.90% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #28      +/-   ##
==========================================
+ Coverage   89.98%   90.68%   +0.70%     
==========================================
  Files           4        4              
  Lines        1048     1041       -7     
  Branches      110      106       -4     
==========================================
+ Hits          943      944       +1     
+ Misses         79       74       -5     
+ Partials       26       23       -3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@timkpaine
timkpaine merged commit fbab3d0 into main Jul 28, 2026
3 of 4 checks passed
@timkpaine
timkpaine deleted the fix-ruff-lint branch July 28, 2026 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant