Skip to content

Issue 531#540

Closed
LLLLimbo wants to merge 8 commits into
dapr:masterfrom
LLLLimbo:issue_531
Closed

Issue 531#540
LLLLimbo wants to merge 8 commits into
dapr:masterfrom
LLLLimbo:issue_531

Conversation

@LLLLimbo

@LLLLimbo LLLLimbo commented May 6, 2021

Copy link
Copy Markdown
Contributor

Description

see #531
Add plugins back in parent pom.

Issue reference

#531

Please reference the issue this PR will close: #[issue number]

Checklist

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

@codecov

codecov Bot commented May 6, 2021

Copy link
Copy Markdown

Codecov Report

Merging #540 (1c7fb22) into master (dfd0d5c) will increase coverage by 0.13%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #540      +/-   ##
============================================
+ Coverage     80.52%   80.65%   +0.13%     
- Complexity      981      982       +1     
============================================
  Files            88       88              
  Lines          3024     3024              
  Branches        331      331              
============================================
+ Hits           2435     2439       +4     
+ Misses          423      420       -3     
+ Partials        166      165       -1     
Impacted Files Coverage Δ Complexity Δ
sdk/src/main/java/io/dapr/utils/Retry.java 75.00% <0.00%> (+25.00%) 2.00% <0.00%> (+1.00%)

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 dfd0d5c...1c7fb22. Read the comment docs.

@artursouza

Copy link
Copy Markdown
Contributor

Thanks for this. I will look at this post v1.1 release since we don't want to risk breaking the CI pipeline on the code freeze date.

@artursouza artursouza self-assigned this May 14, 2021
@artursouza

Copy link
Copy Markdown
Contributor

Please, resolve conflicts. We can add this in this milestone.

@mthmulders

Copy link
Copy Markdown
Contributor

@LLLLimbo, do you intend to continue work on this pull request? If not, how much of this is still relevant after #604?

@mukundansundar

Copy link
Copy Markdown
Contributor

@LLLLimbo is this PR still relevant after #604 ?

@artursouza artursouza closed this Mar 22, 2022
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