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

2 participants