Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

refactor(s3stream): standardize the code style #864

Merged
merged 3 commits into from
Dec 29, 2023
Merged

Conversation

ShadowySpirits
Copy link
Contributor

No description provided.

Copy link

codecov bot commented Dec 28, 2023

Codecov Report

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

Comparison is base (1a1f5ba) 54.98% compared to head (51a29b3) 54.99%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #864      +/-   ##
============================================
+ Coverage     54.98%   54.99%   +0.01%     
- Complexity     1244     1245       +1     
============================================
  Files           133      133              
  Lines          8535     8535              
  Branches        789      789              
============================================
+ Hits           4693     4694       +1     
  Misses         3426     3426              
+ Partials        416      415       -1     
Components Coverage Δ
RocketMQ Broker ∅ <ø> (∅)
RocketMQ Common 5.80% <ø> (ø)
RocketMQ Controller 54.58% <ø> (ø)
RocketMQ Proxy 37.28% <ø> (+0.10%) ⬆️
RocketMQ Store 73.57% <0.00%> (-0.05%) ⬇️
RocketMQ Stream ∅ <ø> (∅)
RocketMQ Metadata 35.44% <ø> (ø)
Files Coverage Δ
.../java/com/automq/rocketmq/store/S3StreamStore.java 0.00% <0.00%> (ø)
...ava/com/automq/rocketmq/store/S3StreamManager.java 0.00% <0.00%> (ø)

... and 2 files with indirect coverage changes

Signed-off-by: SSpirits <admin@lv5.moe>
Signed-off-by: SSpirits <admin@lv5.moe>
@ShadowySpirits ShadowySpirits merged commit f4e37d1 into main Dec 29, 2023
11 of 13 checks passed
@ShadowySpirits ShadowySpirits deleted the code-style branch December 29, 2023 03:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants