Skip to content

#2187: Integrate Ruff - #2202

Open
MariusFreyer wants to merge 2 commits into
devonfw:mainfrom
MariusFreyer:feature/2189-integrate-ruff
Open

#2187: Integrate Ruff#2202
MariusFreyer wants to merge 2 commits into
devonfw:mainfrom
MariusFreyer:feature/2189-integrate-ruff

Conversation

@MariusFreyer

@MariusFreyer MariusFreyer commented Jul 22, 2026

Copy link
Copy Markdown

This PR fixes #2189

Implemented changes:

  • Integrated Ruff

Testing instructions

Please add conscise, understandable instructions on how a reviewer can test/verify the functionality of your contribution here:

  1. Run ide ruff

Checklist for this PR

Make sure everything is checked before merging this PR. For further info please also see
our DoD.

  • When running mvn clean test locally all tests pass and build is successful
  • PR title is of the form #«issue-id»: «brief summary» (e.g. #921: fixed setup.bat). If no issue ID exists, title only.
  • PR top-level comment summarizes what has been done and contains link to addressed issue(s)
  • PR and issue(s) have suitable labels
  • Issue is set to In Progress and assigned to you or there is no issue (might happen for very small PRs)
  • You followed all coding conventions
  • You have added the issue implemented by your PR in CHANGELOG.adoc unless issue is labeled
    with internal
  • You have formulated clear instructions on how to test your contribution under "Testing instructions"

Checklist for tool commandlets

Have you added a new «tool» as commandlet? There are the following additional checks:

  • The tool can be installed automatically (during setup via settings) or via the commandlet call
  • The tool is isolated in its IDEasy project, see Sandbox Principle
  • The new tool is added to the table of tools in LICENSE.asciidoc
  • The new commandlet is a command-wrapper for «tool»
  • Proper help texts for all supported languages are added here
  • The new commandlet installs potential dependencies automatically
  • The variables «TOOL»_VERSION and «TOOL»_EDITION are honored by your commandlet
  • The new commandlet is tested on all platforms it is available for or tested on all platforms that are in scope of the linked issue

@github-project-automation github-project-automation Bot moved this to 🆕 New in IDEasy board Jul 22, 2026
@MariusFreyer MariusFreyer self-assigned this Jul 22, 2026
@MariusFreyer MariusFreyer moved this from 🆕 New to 🏗 In progress in IDEasy board Jul 23, 2026
@coveralls

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 29996096637

Coverage increased (+0.002%) to 72.498%

Details

  • Coverage increased (+0.002%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 8 coverage regressions across 2 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

8 previously-covered lines in 2 files lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/commandlet/CommandletManagerImpl.java 7 91.9%
com/devonfw/tools/ide/version/VersionSegment.java 1 89.5%

Coverage Stats

Coverage Status
Relevant Lines: 16965
Covered Lines: 12828
Line Coverage: 75.61%
Relevant Branches: 7586
Covered Branches: 4971
Branch Coverage: 65.53%
Branches in Coverage %: Yes
Coverage Strength: 3.2 hits per line

💛 - Coveralls

@MariusFreyer MariusFreyer added enhancement New feature or request ready-to-implement uv fast Python package and project manager labels Jul 23, 2026
@MariusFreyer
MariusFreyer force-pushed the feature/2189-integrate-ruff branch from 3b78421 to 27c9f99 Compare July 23, 2026 11:45
@MariusFreyer MariusFreyer moved this from 🏗 In progress to Team Review in IDEasy board Jul 23, 2026
@oanding-blrng oanding-blrng self-assigned this Jul 30, 2026

@oanding-blrng oanding-blrng left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked every point of the checklist and could verify every point. But I'm not able to check off the points in the checklist. Would you @MariusFreyer please go also over the checklist and verify, that all the points are fulfilled?

Besides that I wasn't able to find anything.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request ready-to-implement uv fast Python package and project manager

Projects

Status: Team Review

Development

Successfully merging this pull request may close these issues.

Integrate ruff

3 participants