-
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
gradle-plugin: add task to merge sarif reports #1456
Conversation
### What's done: * Added task to merge SARIF reports
### What's done: * Added task to merge SARIF reports
### What's done: * Added task to merge SARIF reports
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.
Found 15 potential problems in the proposed changes. Check the Files changed tab for more details.
### What's done: * Added task to merge SARIF reports
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.
Found 19 potential problems in the proposed changes. Check the Files changed tab for more details.
### What's done: * Added task to merge SARIF reports
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.
Found 24 potential problems in the proposed changes. Check the Files changed tab for more details.
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.
Found 24 potential problems in the proposed changes. Check the Files changed tab for more details.
### What's done: * Fix test
…' into feature/merge-sarif-reports#1452
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.
Found 24 potential problems in the proposed changes. Check the Files changed tab for more details.
Codecov Report
@@ Coverage Diff @@
## master #1456 +/- ##
============================================
- Coverage 83.27% 82.96% -0.32%
+ Complexity 2569 2567 -2
============================================
Files 110 111 +1
Lines 7661 7695 +34
Branches 2107 2112 +5
============================================
+ Hits 6380 6384 +4
- Misses 387 417 +30
Partials 894 894
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: * Code style
…-reports#1452 # Conflicts: # diktat-gradle-plugin/src/main/kotlin/org/cqfn/diktat/plugin/gradle/DiktatJavaExecTaskBase.kt # diktat-gradle-plugin/src/main/kotlin/org/cqfn/diktat/plugin/gradle/Utils.kt
Please add readme, no need to think about tests :) |
### What's done: * Update README.md * Aggregate only those tasks which have SARIF reporter enabled
…' into feature/merge-sarif-reports#1452
What's done:
This pull request closes #1452