@shollyman review por favor...

@shollyman Friendly nudge, would love to leverage this.

Thank you so much for the PR. Apologies as I somehow didn't see this for the longest time. For the most part the PR looks good but I need to confirm with the team on how to tackle this since we generally try to minimize the number of third party libraries.

@hongalex Hi! Thanks for your reviews. Just added test for escapes characters and rebased to main.

Checks are failing because of a minor formatting issue that running go vet should solve.

Checks are failing because of a minor formatting issue that running go vet should solve.

Thanks! Seems it requires another approval to trigger the workflow:)

One more vet issue, the exported ValidateFilter method needs a comment there.

One more vet issue, the exported ValidateFilter method needs a comment there.

Added the comment. Couldn't succeed in running vet.sh locally. Hopefully, it works this time.

Checked the error message stating that TestUpdateFilter failed due to a bad filter string. Already updated! 🙌

(However, the test TestSubscriptionMessageOrdering still fails on my computer in both the main and this branch. It seems unrelated to this PR.)

I'm so happy this is merged, thank you gcloud team ❤️