Skip to content

#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

#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

@shodiBoy1 shodiBoy1 commented May 25, 2026

This PR fixes #1846

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"

@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

@hohwille hohwille changed the title Feature/1846 skip xattr for non app installs #1846: skip xattr for non app installs May 29, 2026
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.

Mac Gatekeeper workaround causing errors

2 participants