Skip to content

Implemented Replace Matches Functional Operator#114

Merged
copybara-service[bot] merged 2 commits intogoogle:mainfrom
bodiebice:ReplaceMatches
May 28, 2025
Merged

Implemented Replace Matches Functional Operator#114
copybara-service[bot] merged 2 commits intogoogle:mainfrom
bodiebice:ReplaceMatches

Conversation

@bodiebice
Copy link
Copy Markdown
Contributor

I implemented the Replace Matches functional operator.

Copy link
Copy Markdown
Collaborator

@evan-gordon evan-gordon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good so I'll approve! Mind fixing the formatting changes though?

Comment thread interpreter/operator_dispatcher.go
Comment thread tests/enginetests/operator_string_test.go Outdated
@bodiebice bodiebice force-pushed the ReplaceMatches branch 3 times, most recently from e008010 to b094b8e Compare May 6, 2025 21:09
@bodiebice
Copy link
Copy Markdown
Contributor Author

@evan-gordon I think I was able to fix all of the visual issues. Going to look into seeing if linter settings are causing issues

@bodiebice
Copy link
Copy Markdown
Contributor Author

@evan-gordon Should have updated and fixed failing xml test.

@copybara-service copybara-service Bot merged commit 79a0898 into google:main May 28, 2025
5 checks passed
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.

2 participants