Skip to content

[Feature] createSchedule should return schedule's Id#1861

Merged
Technoboy- merged 1 commit intoapache:devfrom
gabry-lab:dev
Jan 20, 2020
Merged

[Feature] createSchedule should return schedule's Id#1861
Technoboy- merged 1 commit intoapache:devfrom
gabry-lab:dev

Conversation

@gabry-lab
Copy link
Copy Markdown
Member

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1861 into dev will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #1861      +/-   ##
==========================================
- Coverage   22.47%   22.46%   -0.01%     
==========================================
  Files         296      296              
  Lines       14181    14182       +1     
  Branches     2342     2342              
==========================================
- Hits         3187     3186       -1     
- Misses      10637    10640       +3     
+ Partials      357      356       -1
Impacted Files Coverage Δ
...dolphinscheduler/api/service/SchedulerService.java 0.42% <0%> (-0.01%) ⬇️
...scheduler/alert/template/AlertTemplateFactory.java 42.85% <0%> (-14.29%) ⬇️

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 9549910...c125e3f. Read the comment docs.

@sonarqubecloud
Copy link
Copy Markdown

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

@gabry-lab gabry-lab requested a review from davidzollo January 20, 2020 05:25
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

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