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

Update README and move sarif utils into common module #477

Merged
merged 6 commits into from
Jan 13, 2023

Conversation

kgevorkyan
Copy link
Member

What's done:

  • Update README and move sarif utils into common module

### What's done:
* WIP
### What's done:
* WIP
### What's done:
* WIP
### What's done:
* WIP
### What's done:
* WIP
### What's done:
* WIP
@kgevorkyan kgevorkyan marked this pull request as ready for review January 13, 2023 09:40
@kgevorkyan kgevorkyan linked an issue Jan 13, 2023 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Jan 13, 2023

Codecov Report

Merging #477 (d155eae) into main (69b23ee) will decrease coverage by 0.55%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main     #477      +/-   ##
============================================
- Coverage     47.57%   47.02%   -0.56%     
+ Complexity      153      152       -1     
============================================
  Files            53       53              
  Lines          1814     1814              
  Branches        313      313              
============================================
- Hits            863      853      -10     
- Misses          821      831      +10     
  Partials        130      130              
Flag Coverage Δ
unittests 47.02% <0.00%> (-0.56%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
.../com/saveourtool/save/core/utils/SarifFileUtils.kt 0.00% <0.00%> (ø)
...tlin/com/saveourtool/save/plugins/fix/FixPlugin.kt 56.62% <0.00%> (-1.05%) ⬇️
...om/saveourtool/save/plugins/fix/FixPluginConfig.kt 43.58% <0.00%> (-0.86%) ⬇️
...tool/save/plugin/warn/sarif/SarifWarningAdapter.kt 58.33% <ø> (ø)
...rtool/save/plugin/warn/utils/WarningsExtraction.kt 36.00% <0.00%> (+0.70%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@github-actions
Copy link

JUnit Tests (Linux, EnricoMi/publish-unit-test-result-action@v1)

241 tests  ±0   217 ✔️ ±0   9m 6s ⏱️ - 3m 16s
  48 suites ±0     24 💤 ±0 
  48 files   ±0       0 ±0 

Results for commit d155eae. ± Comparison against base commit 69b23ee.

@github-actions
Copy link

JUnit Tests (Windows, EnricoMi/publish-unit-test-result-action@v1)

241 tests  ±0   217 ✔️ ±0   10m 10s ⏱️ -1s
  48 suites ±0     24 💤 ±0 
  48 files   ±0       0 ±0 

Results for commit d155eae. ± Comparison against base commit 69b23ee.

@github-actions
Copy link

JUnit Tests (macOS, EnricoMi/publish-unit-test-result-action@v1)

241 tests  ±0   217 ✔️ ±0   13m 2s ⏱️ +47s
  48 suites ±0     24 💤 ±0 
  48 files   ±0       0 ±0 

Results for commit d155eae. ± Comparison against base commit 69b23ee.

@kgevorkyan kgevorkyan merged commit e5ced44 into main Jan 13, 2023
@kgevorkyan kgevorkyan deleted the feature/sarif-readme branch January 13, 2023 10:21
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.

Update readme of FixPlugin with information about sarif mode
2 participants