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

Log producer should have qualifier #4

Merged
merged 4 commits into from
Aug 31, 2018

Conversation

djr4488
Copy link
Owner

@djr4488 djr4488 commented Aug 31, 2018

No description provided.

djr4488 added 4 commits August 31, 2018 07:25
 - added qualifier for logs
 - added qualifier for logs
 - added qualifier for logs
 - added qualifier for logs
@codecov-io
Copy link

codecov-io commented Aug 31, 2018

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master       #4   +/-   ##
=========================================
  Coverage     92.26%   92.26%           
  Complexity       88       88           
=========================================
  Files            15       15           
  Lines           194      194           
  Branches         18       18           
=========================================
  Hits            179      179           
  Misses            9        9           
  Partials          6        6
Impacted Files Coverage Δ Complexity Δ
.../java/org/djr/cdi/properties/PropertyResolver.java 88.57% <ø> (ø) 16 <0> (ø) ⬇️
src/main/java/org/djr/cdi/logs/LoggerProducer.java 100% <100%> (ø) 2 <1> (ø) ⬇️

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 ca18183...e830a4f. Read the comment docs.

@djr4488 djr4488 merged commit 1e1eaab into master Aug 31, 2018
@coveralls
Copy link

Pull Request Test Coverage Report for Build 35

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.361%

Totals Coverage Status
Change from base Build 25: 0.0%
Covered Lines: 185
Relevant Lines: 194

💛 - Coveralls

@djr4488 djr4488 deleted the log-producer-should-have-qualifier branch August 31, 2018 13:11
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