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

Write all crit events to cerr #696

Merged
merged 3 commits into from
Mar 28, 2024
Merged

Conversation

drbasic
Copy link
Collaborator

@drbasic drbasic commented Mar 11, 2024

No description provided.

komarevtsev-d
komarevtsev-d previously approved these changes Mar 11, 2024
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 7db14ef.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
4791 4716 0 0 1 74

komarevtsev-d
komarevtsev-d previously approved these changes Mar 14, 2024
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 102627c.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
4791 4719 0 0 0 72

@drbasic drbasic force-pushed the users/drbasic/always-log-critevent branch from 102627c to 1c2d520 Compare March 28, 2024 09:01
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 1c2d520.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5065 5061 0 0 2 2

@drbasic drbasic merged commit d3cd2bf into main Mar 28, 2024
7 checks passed
@drbasic drbasic deleted the users/drbasic/always-log-critevent branch March 28, 2024 13:05
qkrorlqr pushed a commit that referenced this pull request Nov 28, 2024
* Write all crit events to cerr

* Make writing almost atomic

* Use global logger
qkrorlqr added a commit that referenced this pull request Nov 28, 2024
* Write all crit events to cerr (#696)

* Write all crit events to cerr

* Make writing almost atomic

* Use global logger

* fixed build after merge

---------

Co-authored-by: drbasic <[email protected]>
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