-
Notifications
You must be signed in to change notification settings - Fork 168
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
CI fix for topic reflection. #1448
Conversation
04349d4
to
ba1420c
Compare
Signed-off-by: Hermann0222 <yukang.wei@emqx.io>
Signed-off-by: Hermann0222 <yukang.wei@emqx.io>
Signed-off-by: Hermann0222 <yukang.wei@emqx.io>
Signed-off-by: Hermann0222 <yukang.wei@emqx.io>
Signed-off-by: Hermann0222 <yukang.wei@emqx.io>
ba1420c
to
64386da
Compare
Codecov ReportPatch coverage is
📢 Thoughts on this report? Let us know!. |
Signed-off-by: Hermann0222 <yukang.wei@emqx.io>
c578d23
to
7b5a68f
Compare
More code needs to be adjusted for topic reflection, like rest api bridge sub & unsub. Before that tests for those are not available, so the codecov may go down. |
719652d
to
038274b
Compare
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.
The code coverage has fallen below 50%!!!
This PR is meant to fix the CI issue. As for the issue mentioned above, there will be another PR to fix it. |
a434a6b
to
26c4dae
Compare
Signed-off-by: Hermann0222 <yukang.wei@emqx.io>
26c4dae
to
b36e8fd
Compare
THE another PR --> #1455 |
No description provided.