Skip to content

Fix lint errors#35

Merged
timkpaine merged 1 commit into
mainfrom
tkp/ruff
Jul 23, 2026
Merged

Fix lint errors#35
timkpaine merged 1 commit into
mainfrom
tkp/ruff

Conversation

@timkpaine

@timkpaine timkpaine commented Jul 23, 2026

Copy link
Copy Markdown
Member

Updates code for the latest Ruff diagnostics without changing lint configuration. make lint passes.

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

Copy link
Copy Markdown
Contributor

Test Results

12 tests  ±0   10 ✅ ±0   0s ⏱️ ±0s
 1 suites ±0    2 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 223c628. ± Comparison against base commit a8b49ff.

@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 85.35%. Comparing base (a8b49ff) to head (223c628).

Files with missing lines Patch % Lines
fsspec_python/importer.py 83.33% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #35   +/-   ##
=======================================
  Coverage   85.35%   85.35%           
=======================================
  Files           5        5           
  Lines         157      157           
  Branches       27       27           
=======================================
  Hits          134      134           
  Misses         12       12           
  Partials       11       11           

☔ 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 8605cd8 into main Jul 23, 2026
5 of 6 checks passed
@timkpaine
timkpaine deleted the tkp/ruff branch July 23, 2026 22:38
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