Skip to content

Comments

fix: Remove git stash from pre-commit hook to prevent merge conflicts#23617

Merged
Artur- merged 2 commits intomainfrom
fix-hook
Feb 23, 2026
Merged

fix: Remove git stash from pre-commit hook to prevent merge conflicts#23617
Artur- merged 2 commits intomainfrom
fix-hook

Conversation

@Artur-
Copy link
Member

@Artur- Artur- commented Feb 21, 2026

The stash push/pop cycle caused merge conflicts when Spotless reformatted staged files, leaving the working tree in a broken state requiring manual resolution. The trade-off is that partial staging (git add -p) may now include unstaged formatting changes, which is cosmetic and recoverable.

The stash push/pop cycle caused merge conflicts when Spotless reformatted
staged files, leaving the working tree in a broken state requiring manual
resolution. The trade-off is that partial staging (git add -p) may now
include unstaged formatting changes, which is cosmetic and recoverable.
@github-actions
Copy link

github-actions bot commented Feb 21, 2026

Test Results

 1 365 files  ±0   1 365 suites  ±0   1h 21m 17s ⏱️ - 1m 0s
 9 717 tests ±0   9 650 ✅ ±0  67 💤 ±0  0 ❌ ±0 
10 190 runs  ±0  10 114 ✅ ±0  76 💤 ±0  0 ❌ ±0 

Results for commit 43196d9. ± Comparison against base commit 6a01d87.

♻️ This comment has been updated with latest results.

@Artur- Artur- requested a review from heruan February 21, 2026 17:24
@sonarqubecloud
Copy link

@Artur- Artur- merged commit 53aa69b into main Feb 23, 2026
31 checks passed
@Artur- Artur- deleted the fix-hook branch February 23, 2026 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants