Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#531: prevent logging for processable commandlets #541

Conversation

hohwille
Copy link
Member

fixes #531
fixes #511

@hohwille hohwille added enhancement New feature or request test related to testing and QA bugfix logging core FileAccess, ProcessUtil, IdeContext, etc. labels Aug 18, 2024
@hohwille hohwille added this to the release:2024.09.001 milestone Aug 18, 2024
@hohwille hohwille self-assigned this Aug 18, 2024
@coveralls
Copy link
Collaborator

coveralls commented Aug 18, 2024

Pull Request Test Coverage Report for Build 10466714699

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 102 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.05%) to 64.178%

Files with Coverage Reduction New Missed Lines %
com/devonfw/tools/ide/context/IdeContextConsole.java 7 20.69%
com/devonfw/tools/ide/context/AbstractIdeContext.java 95 62.06%
Totals Coverage Status
Change from base Build 10457899767: 0.05%
Covered Lines: 5659
Relevant Lines: 8486

💛 - Coveralls

@hohwille hohwille merged commit b731286 into devonfw:main Aug 20, 2024
4 checks passed
@hohwille hohwille added the reviewed Marks PRs that have been presented in the sprint-review meeting or that do not need to be presented. label Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix core FileAccess, ProcessUtil, IdeContext, etc. enhancement New feature or request logging reviewed Marks PRs that have been presented in the sprint-review meeting or that do not need to be presented. test related to testing and QA
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

EnvironmentCommandlet can log warnings into the middle of environment variables Consolidate test context
3 participants