Skip to content

Conversation

mstv
Copy link
Member

@mstv mstv commented Jan 2, 2025

Avoids double update of git-grep if search text matches a history entry

Proposed changes

FileStatusList:

  • Temporarily remove cboFindInCommitFilesGitGrep_SelectedIndexChanged in AddToSearchFilter
  • Declare sender nullable for all event handlers

Screenshots

N/A

Test methodology

  • manual

Merge strategy

I agree that the maintainer squash merge this PR (if the commit message is clear).


✒️ I contribute this code under The Developer Certificate of Origin.

@mstv mstv self-assigned this Jan 2, 2025
@mstv mstv merged commit afdfac6 into gitextensions:master Jan 3, 2025
4 checks passed
@mstv mstv deleted the fix/git_grep_update branch January 3, 2025 18:39
@mstv mstv added this to the v5.2 milestone Jan 8, 2025
@mstv mstv removed the status: ready label Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants