Skip to content

chore(lint): apply ruff's safe autofixes - #771

Merged
petercorke merged 1 commit into
mainfrom
chore/ruff-autofix
Oct 14, 2026
Merged

petercorke merged 1 commit into
mainfrom
chore/ruff-autofix

Conversation

@petercorke

Copy link
Copy Markdown
Owner

One mechanical pass of ruff check --fix with F401 (unused imports), I001 (import order), UP037 (quoted annotations), UP032 (f-strings) and RUF100 (unused noqa): 139 files, about 570 findings.

Left alone on purpose: every __init__.py (import order and re-exports can matter there) and bin/rtbtool.py (its star imports and 'unused' imports are the IPython session namespace). Nothing else is mixed in.

mypy clean; the local test run is unchanged (the same 19 compiled-extension tests fail in a source worktree as on main).

🤖 Generated with Claude Code

Unused imports, import order, quoted annotations, f-strings and unused
noqa comments, from ruff check --fix with F401, I001, UP037, UP032 and
RUF100.  No __init__.py files, and not rtbtool.py, whose imports are its
interactive namespace.  mypy clean, tests unchanged.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@codacy-production

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 1 high

Alerts:
⚠ 1 issue (≤ 0 issues of at least minor severity)

Results:
1 new issue

Category Results
Security 1 high

View in Codacy

🟢 Metrics 0 complexity · -9 duplication

Metric Results
Complexity 0
Duplication -9

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@codecov

codecov Bot commented Oct 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 353 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (2de1e3e) to head (5c4f5c4).

Files with missing lines Patch % Lines
src/roboticstoolbox/ets/ETS.py 0.00% 17 Missing ⚠️
src/roboticstoolbox/ets/fknm.py 0.00% 16 Missing ⚠️
src/roboticstoolbox/robot/DHRobot.py 0.00% 15 Missing ⚠️
src/roboticstoolbox/robot/BaseRobot.py 0.00% 13 Missing ⚠️
src/roboticstoolbox/robot/Link.py 0.00% 12 Missing ⚠️
src/roboticstoolbox/robot/IK.py 0.00% 10 Missing ⚠️
src/roboticstoolbox/backends/PyPlot/PyPlot.py 0.00% 9 Missing ⚠️
src/roboticstoolbox/models/URDF/URDFRobot.py 0.00% 9 Missing ⚠️
src/roboticstoolbox/robot/Dynamics.py 0.00% 9 Missing ⚠️
src/roboticstoolbox/backends/PyPlot/PyPlot2.py 0.00% 8 Missing ⚠️
... and 83 more
Additional details and impacted files
@@          Coverage Diff           @@
##            main    #771    +/-   ##
======================================
  Coverage   0.00%   0.00%            
======================================
  Files        143     143            
  Lines      14785   14667   -118     
======================================
+ Misses     14785   14667   -118     

☔ 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.

@petercorke
petercorke merged commit f28437f into main Oct 14, 2026
25 of 26 checks passed
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