-
Notifications
You must be signed in to change notification settings - Fork 4
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
Improve performance a lot #13
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much for improving the performance of this function. This is an huge gain 🙏 🎉
I only have two a minor comments other than that this looks great!
Co-authored-by: Lukas Geiger <lukas.geiger94@gmail.com>
Thank you @lgeiger, I've just updated the solution based on your review :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! Thank you so much for the PR 🚀
Turns out I don't have my npm 2FA authenticator with me at the moment I will release a new patch tonight. |
Published as v1.3.1 🚀 Thanks again for the PR! |
Thanks a lot for the quick release! 🚀 |
We have experienced performance problems with the carriage returns escaping in kubeshop/testkube#3522. To solve the problem, I've improved the performance of this library.
Benchmarks
Before
After