dajiaohuang commented on PR #9110: URL: https://github.com/apache/hop/pull/9110#issuecomment-6453584399
Thanks for the detailed review. I pushed commit `1b4d9135346f0cbc51d7d8b7ee9e737c0e23e9c3` to the existing PR branch to check path-scoped local changes before the confirmation dialog for file revert/cherry-pick, with a localized warning; the restore helper still checks as a backstop. The tests now cover another dirty path, local edits on a file the selected commit added, an untracked file where a revert would restore a deleted path, and the staged-change exception message. I also made the existing dirty-repository warnings for commit-level revert/cherry-pick visible. Validation: the targeted Maven build completed successfully; `UIGitTest` ran 66 tests with 0 failures/errors/skips. `git diff --check` passed. Compilation reported existing warnings; CI was not queried. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
