-
Notifications
You must be signed in to change notification settings - Fork 39
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
code coverage in maven plugin #644
Conversation
### What's done: * Fixed pom.xml
Codecov Report
@@ Coverage Diff @@
## master #644 +/- ##
============================================
+ Coverage 78.91% 79.07% +0.15%
+ Complexity 1789 1787 -2
============================================
Files 88 88
Lines 4696 4697 +1
Branches 1506 1506
============================================
+ Hits 3706 3714 +8
+ Misses 298 288 -10
- Partials 692 695 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
### What's done: * WIP: GH packages, maven example
### What's done: * maven tests
### What's done: * Fixed code in examples
### What's done: * Fixed code in examples
### What's done: * Added jacoco agent via .mvn/jvm.config
### What's done: * Added jacoco agent via .mvn/jvm.config
### What's done: * Tests on diktat:fix
### What's done: * Changed !== to !=
diktat-maven-plugin/src/test/kotlin/org/cqfn/diktat/plugin/maven/DiktatBaseMojoTest.kt
Show resolved
Hide resolved
diktat-maven-plugin/src/main/kotlin/org/cqfn/diktat/plugin/maven/DiktatBaseMojo.kt
Show resolved
Hide resolved
### What's done: * resolveConfig returns non-nullable
### What's done: * resolveConfig returns non-nullable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
now looks good, thanks!
What's done:
This pull request closes #631