#2190: IDEasy destroys python installation - #2212
Conversation
Coverage Report for CI Build 30344918220Coverage increased (+0.03%) to 72.628%Details
Uncovered ChangesNo uncovered changes found. Coverage Regressions5 previously-covered lines in 2 files lost coverage.
Coverage Stats💛 - Coveralls |
hohwille
left a comment
There was a problem hiding this comment.
Since this is still in team-review, I should actually be patient but since I already had a look.
The bug comes from this location:
IDEasy/cli/src/main/java/com/devonfw/tools/ide/tool/LocalToolCommandlet.java
Lines 197 to 200 in c6fcf9b
(you can easily use the GitHub search to find such code location from the log message).
You are not fixing this in your current PR.
15cad77 to
dd45e91
Compare
@hohwille Thanks for the review. I investigated your suggestion, identified the root cause, and fixed the issue in the latest commit. I've also updated the problem analysis and the testing instructions above. Please take another look and let me know your feedback. Sorry for the delayed update, and thanks again! |
This PR fixes #2190
Implemented changes:
Testing instructions
ide install pythononce (orSpyder) and delete the file$[IDE_HOME]/software/python/.ide.software.version.Result:
$IDE_HOME/software/python/.ide.software.version.Checklist for this PR
Make sure everything is checked before merging this PR. For further info please also see
our DoD.
mvn clean testlocally all tests pass and build is successful#«issue-id»: «brief summary»(e.g.#921: fixed setup.bat). If no issue ID exists, title only.In Progressand assigned to you or there is no issue (might happen for very small PRs)with
internal