Skip to content

Feature/1846 skip xattr for non app installs#1970

Draft
shodiBoy1 wants to merge 3 commits into
devonfw:mainfrom
shodiBoy1:feature/1846-skip-xattr-for-non-app-installs
Draft

Feature/1846 skip xattr for non app installs#1970
shodiBoy1 wants to merge 3 commits into
devonfw:mainfrom
shodiBoy1:feature/1846-skip-xattr-for-non-app-installs

Conversation

@shodiBoy1
Copy link
Copy Markdown
Contributor

This PR fixes TodoIssueID

Implemented changes:

  • Change 1

Testing instructions

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


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 May 25, 2026
@shodiBoy1 shodiBoy1 self-assigned this May 25, 2026
@shodiBoy1 shodiBoy1 added macOS specific for Apple MacOS bugfix PR that fixes a bug issue and removed macOS specific for Apple MacOS labels May 25, 2026
@coveralls
Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 26477587146

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage increased (+0.02%) to 70.991%

Details

  • Coverage increased (+0.02%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 13 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

13 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/os/MacOsHelper.java 13 77.31%

Coverage Stats

Coverage Status
Relevant Lines: 15630
Covered Lines: 11577
Line Coverage: 74.07%
Relevant Branches: 6970
Covered Branches: 4467
Branch Coverage: 64.09%
Branches in Coverage %: Yes
Coverage Strength: 3.13 hits per line

💛 - Coveralls

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

Labels

bugfix PR that fixes a bug issue

Projects

Status: 🆕 New

Development

Successfully merging this pull request may close these issues.

2 participants