Skip to content
Snippets Groups Projects

search: Handle ()() in autofix

Created by: keegancsmith

Noticed this flakey test failure in CI. It is a legitimate issue were we do not properly escape ()(). It escaped it as \(\)(). This is due to the replacement regex checking for the preceding character being \, which means when ReplaceAll runs after replacing the first () the regex won't match since ^ means beginning of string, not match.

I wasn't sure how to make this happen in a regex, so wrote a function implementing the functionality.

Merge request reports

Approval is optional

Merged by avatar (Jul 30, 2025 11:05pm UTC)

Merge details

  • Changes merged into master with 89e1d644.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading