devonfw / IDEasy

Tool to automate the setup and updates of a development environment for any project (Successor of devonfw-ide).
Apache License 2.0
8 stars 20 forks source link

#171: support for multiple VariableSyntax styles #407

Closed hohwille closed 3 months ago

hohwille commented 3 months ago

fixes #171

coveralls commented 3 months ago

Pull Request Test Coverage Report for Build 9613602697

Details


Files with Coverage Reduction New Missed Lines %
com/devonfw/tools/ide/merge/FileMerger.java 4 33.33%
com/devonfw/tools/ide/environment/EnvironmentVariables.java 7 81.36%
com/devonfw/tools/ide/merge/xmlmerger/XmlMerger.java 10 77.78%
com/devonfw/tools/ide/merge/JsonMerger.java 12 77.33%
com/devonfw/tools/ide/merge/PropertiesMerger.java 15 41.67%
com/devonfw/tools/ide/environment/AbstractEnvironmentVariables.java 20 78.24%
<!-- Total: 68 -->
Totals Coverage Status
Change from base Build 9610011440: 0.01%
Covered Lines: 4972
Relevant Lines: 7883

💛 - Coveralls
coveralls commented 3 months ago

Pull Request Test Coverage Report for Build 9615912827

Details


Files with Coverage Reduction New Missed Lines %
com/devonfw/tools/ide/merge/FileMerger.java 4 33.33%
com/devonfw/tools/ide/environment/EnvironmentVariables.java 7 81.36%
com/devonfw/tools/ide/context/IdeContext.java 7 45.28%
com/devonfw/tools/ide/merge/xmlmerger/XmlMerger.java 10 77.78%
com/devonfw/tools/ide/merge/JsonMerger.java 12 77.33%
com/devonfw/tools/ide/merge/PropertiesMerger.java 15 41.67%
com/devonfw/tools/ide/environment/AbstractEnvironmentVariables.java 20 78.24%
com/devonfw/tools/ide/step/Step.java 20 36.36%
com/devonfw/tools/ide/tool/mvn/Mvn.java 35 41.54%
<!-- Total: 130 -->
Totals Coverage Status
Change from base Build 9610011440: -0.06%
Covered Lines: 4981
Relevant Lines: 7905

💛 - Coveralls
hohwille commented 3 months ago

@slskiba did some review and could not find concerns but also not verify all changes as reported today in daily meeting. I will merge this today as we need this feature for https://github.com/devonfw/ide-settings/pull/51.

coveralls commented 3 months ago

Pull Request Test Coverage Report for Build 9660943470

Details


Files with Coverage Reduction New Missed Lines %
com/devonfw/tools/ide/merge/FileMerger.java 4 33.33%
com/devonfw/tools/ide/environment/EnvironmentVariables.java 7 81.36%
com/devonfw/tools/ide/context/IdeContext.java 7 45.28%
com/devonfw/tools/ide/merge/xmlmerger/XmlMerger.java 10 77.78%
com/devonfw/tools/ide/merge/JsonMerger.java 12 77.33%
com/devonfw/tools/ide/merge/PropertiesMerger.java 15 41.67%
com/devonfw/tools/ide/environment/AbstractEnvironmentVariables.java 20 78.24%
com/devonfw/tools/ide/step/Step.java 20 36.36%
com/devonfw/tools/ide/tool/mvn/Mvn.java 35 41.54%
<!-- Total: 130 -->
Totals Coverage Status
Change from base Build 9658247376: -0.06%
Covered Lines: 4985
Relevant Lines: 7942

💛 - Coveralls