Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

PSTGRSCROC-135 #430

Merged
merged 1 commit into from Dec 6, 2019
Merged

PSTGRSCROC-135 #430

merged 1 commit into from Dec 6, 2019

Conversation

mgramin
Copy link
Contributor

@mgramin mgramin commented Dec 5, 2019

No description provided.

@codecov-io
Copy link

codecov-io commented Dec 5, 2019

Codecov Report

Merging #430 into master will decrease coverage by 0.63%.
The diff coverage is 83.33%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #430      +/-   ##
============================================
- Coverage     73.96%   73.32%   -0.64%     
  Complexity      212      212              
============================================
  Files            45       45              
  Lines           626      626              
  Branches         61       60       -1     
============================================
- Hits            463      459       -4     
- Misses          118      122       +4     
  Partials         45       45
Impacted Files Coverage Δ Complexity Δ
...sqlboot/model/resourcetype/wrappers/RestWrapper.kt 52.38% <83.33%> (-19.05%) 5 <4> (ø)

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 cd7c656...7a89e15. Read the comment docs.

@mgramin
Copy link
Contributor Author

mgramin commented Dec 6, 2019

@rultor merge

@rultor
Copy link
Collaborator

rultor commented Dec 6, 2019

@rultor merge

@mgramin OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit 7a89e15 into master Dec 6, 2019
@rultor
Copy link
Collaborator

rultor commented Dec 6, 2019

@rultor merge

@mgramin Done! FYI, the full log is here (took me 7min)

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.

None yet

3 participants