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

Replace all nng_fatal with NANO_NNG_FATAL #1516

Merged
merged 4 commits into from Nov 7, 2023
Merged

Replace all nng_fatal with NANO_NNG_FATAL #1516

merged 4 commits into from Nov 7, 2023

Conversation

RanMaoyi
Copy link
Contributor

@RanMaoyi RanMaoyi commented Nov 6, 2023

For record fatal log in nanomq log

Signed-off-by: Moi Ran <maoyi.ran@emqx.io>
Signed-off-by: Moi Ran <maoyi.ran@emqx.io>
Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Attention: 98 lines in your changes are missing coverage. Please review.

Files Coverage Δ
nanomq/conf_api.c 40.19% <0.00%> (ø)
nanomq/aws_bridge.c 80.82% <0.00%> (+1.55%) ⬆️
nanomq/cmd_proc.c 11.38% <0.00%> (ø)
nanomq/webhook_inproc.c 75.75% <0.00%> (ø)
nanomq/bridge.c 59.50% <0.00%> (ø)
nanomq/apps/broker.c 53.01% <0.00%> (ø)
nanomq/web_server.c 71.30% <0.00%> (ø)

... and 2 files with indirect coverage changes

📢 Thoughts on this report? Let us know!

Signed-off-by: jaylin <jaylin@emqx.io>
@JaylinYu JaylinYu merged commit a902f2a into master Nov 7, 2023
5 of 7 checks passed
@JaylinYu JaylinYu deleted the moi_nng_fatal branch November 7, 2023 11:10
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.

None yet

5 participants