Skip to content

kaiyuanke-2301210357: fix了一些warning & 对一些语句做了简化#8237

Closed
Mrhorse99 wants to merge 6 commits intoapache:developfrom
Mrhorse99:develop
Closed

kaiyuanke-2301210357: fix了一些warning & 对一些语句做了简化#8237
Mrhorse99 wants to merge 6 commits intoapache:developfrom
Mrhorse99:develop

Conversation

@Mrhorse99
Copy link
Contributor

Which Issue(s) This PR Fixes

Fixes #issue_id

Brief Description

How Did You Test This Change?

Mrhorse99 and others added 4 commits June 1, 2024 20:29
@Mrhorse99 Mrhorse99 changed the title 开源课作业-2301210357: fix了一些warning & 对一些语句做了简化 kaiyuanke-2301210357: fix了一些warning & 对一些语句做了简化 Jun 1, 2024
@RongtongJin
Copy link
Contributor

RongtongJin commented Jun 1, 2024

@Mrhorse99 It would be better to submit a relevant issue first.

@codecov-commenter
Copy link

codecov-commenter commented Jun 1, 2024

Codecov Report

Attention: Patch coverage is 0% with 12 lines in your changes are missing coverage. Please review.

Project coverage is 42.90%. Comparing base (144b22b) to head (044ac92).

Files Patch % Lines
...ocketmq/broker/processor/AdminBrokerProcessor.java 0.00% 5 Missing and 4 partials ⚠️
...apache/rocketmq/broker/slave/SlaveSynchronize.java 0.00% 2 Missing ⚠️
...g/apache/rocketmq/srvutil/AclFileWatchService.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8237      +/-   ##
=============================================
- Coverage      42.92%   42.90%   -0.03%     
+ Complexity     10383    10371      -12     
=============================================
  Files           1271     1271              
  Lines          88785    88775      -10     
  Branches       11408    11404       -4     
=============================================
- Hits           38112    38087      -25     
- Misses         45973    45977       +4     
- Partials        4700     4711      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Mrhorse99 Mrhorse99 closed this by deleting the head repository Jun 9, 2024
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