Skip to content

Adapting partial code(file name start with B) to the sonar cloud rules#1915

Merged
davidzollo merged 2 commits intoapache:devfrom
gabry-lab:fixSonarB
Feb 9, 2020
Merged

Adapting partial code(file name start with B) to the sonar cloud rules#1915
davidzollo merged 2 commits intoapache:devfrom
gabry-lab:fixSonarB

Conversation

@gabry-lab
Copy link
Copy Markdown
Member

Adapting partial code(file name start with B) to the sonar cloud rules

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1915 into dev will decrease coverage by 0.02%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #1915      +/-   ##
==========================================
- Coverage   24.88%   24.85%   -0.03%     
==========================================
  Files         298      298              
  Lines       14388    14387       -1     
  Branches     2371     2371              
==========================================
- Hits         3580     3576       -4     
- Misses      10359    10363       +4     
+ Partials      449      448       -1
Impacted Files Coverage Δ
...olphinscheduler/api/controller/BaseController.java 3.79% <0%> (+0.13%) ⬆️
...er/common/utils/placeholder/BusinessTimeUtils.java 78.57% <0%> (-13.1%) ⬇️
...er/server/worker/task/AbstractCommandExecutor.java 40.5% <0%> (-2%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bae622f...3704823. Read the comment docs.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Feb 9, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Copy Markdown
Contributor

@Technoboy- Technoboy- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Copy link
Copy Markdown
Contributor

@davidzollo davidzollo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@davidzollo davidzollo merged commit b77dddf into apache:dev Feb 9, 2020
@gabry-lab gabry-lab deleted the fixSonarB branch February 9, 2020 13:59
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.

4 participants