Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Wrap long values into BStr for warn logs #334

Merged
merged 1 commit into from
Aug 31, 2021
Merged

Wrap long values into BStr for warn logs #334

merged 1 commit into from
Aug 31, 2021

Conversation

shekhirin
Copy link
Contributor

Resolves #263

@shekhirin shekhirin changed the title chore(update): wrap long values into BStr for warn logs Wrap long values into BStr for warn logs Aug 31, 2021
@shekhirin shekhirin mentioned this pull request Aug 31, 2021
Closed
Kerollmops
Kerollmops approved these changes Aug 31, 2021
View reviewed changes
Copy link
Member

@Kerollmops Kerollmops left a comment

Choose a reason for hiding this comment

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

Yup, that's indeed the idea! Thank you again Alexey ❤️
bors merge

@bors
Copy link
Contributor

bors bot commented Aug 31, 2021

Build succeeded:

@bors bors bot merged commit d6bba06 into meilisearch:main Aug 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Kerollmops Kerollmops Kerollmops approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Unexpected log output
2 participants
@shekhirin @Kerollmops