Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optimise text_format:escape_label_value #160

Merged
merged 1 commit into from
Nov 20, 2023

Conversation

gomoripeti
Copy link
Contributor

Don't create a new binary if there is no special character to escape.

This PR is based on #158 to preserve attribution of the author of that change.

@deadtrickster
Copy link
Owner

conflict :-(

@gomoripeti
Copy link
Contributor Author

rebased to resolve the conflict

@ikavgo
Copy link
Collaborator

ikavgo commented Nov 17, 2023

Merging is blocked
The base branch requires all commits to be signed. 🤷‍♂️

@gomoripeti
Copy link
Contributor Author

o wow, I have to "Learn more about singing commits" and get back to you

Don't create a new binary if there is no special character to escape.
@gomoripeti
Copy link
Contributor Author

pushed a signed commit now

@deadtrickster deadtrickster merged commit 9186c8c into deadtrickster:master Nov 20, 2023
7 checks passed
@deadtrickster
Copy link
Owner

Great! Thank you

@gomoripeti gomoripeti deleted the escape_label_opt branch November 20, 2023 16:51
@gomoripeti
Copy link
Contributor Author

would you mind tagging a new version including this change, so it can be included in RabbitMQ (used by this PR rabbitmq/rabbitmq-server#9656)

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