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.

Sign up for GitHub

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

Jump to bottom

Added flush logs on buffer size condition #233

Merged

tshemsedinov merged 2 commits into metarhia:master from RohovAlex:fix-flush-on-buffer-size

Oct 17, 2023

Contributor

RohovAlex commented Oct 16, 2023 •

edited

Loading

Added functionality to flush logs on buffer size condition.

tests and linter show no problems (npm t)
tests are added/updated for bug fixes and new features
code is properly formatted (npm run fmt)
description of changes is added in CHANGELOG.md
update .d.ts typings

Contributor Author

RohovAlex commented Oct 16, 2023

@tshemsedinov please review.

tshemsedinov force-pushed the master branch from 51731d7 to 26c19c6 Compare

October 17, 2023 16:11

tshemsedinov approved these changes

View reviewed changes

RohovAlex added 2 commits

October 17, 2023 19:27


          added flush logs on buffer size condition

722cfa1


          add buffer length field to the typings

1b879de

tshemsedinov changed the title ~~added flush logs on buffer size condition~~ Added flush logs on buffer size condition

tshemsedinov merged commit 0d3148d into metarhia:master

6 checks passed

tshemsedinov mentioned this pull request

Support options.writeBuffer #206

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet