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

5: Added sonar check to build job. #17

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

ingokuba
Copy link
Member

@ingokuba ingokuba commented Aug 30, 2018

using sonar in sonarcloud
Closes #5

@ingokuba ingokuba added the enhancement New feature or request label Aug 30, 2018
@ingokuba ingokuba self-assigned this Aug 30, 2018
@ingokuba
Copy link
Member Author

@ingokuba ingokuba force-pushed the feature/#5/addSonar branch 13 times, most recently from 3d97125 to 2b1dbc9 Compare August 30, 2018 11:46
@intensiongmbh intensiongmbh deleted a comment from codecov bot Aug 30, 2018
@codecov
Copy link

codecov bot commented Aug 30, 2018

Codecov Report

Merging #17 into master will decrease coverage by 0.54%.
The diff coverage is 5.55%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
- Coverage   28.79%   28.24%   -0.55%     
==========================================
  Files          26       26              
  Lines         573      577       +4     
  Branches       62       61       -1     
==========================================
- Hits          165      163       -2     
- Misses        401      407       +6     
  Partials        7        7
Impacted Files Coverage Δ
...ava/de/intension/lizzy/plugin/dialogs/Dialogs.java 0% <0%> (ø) ⬆️
...intension/lizzy/plugin/dialogs/PasswordDialog.java 0% <0%> (ø) ⬆️
...zzy/plugin/provider/SecureStorageNodeProvider.java 0% <0%> (ø) ⬆️
...a/de/intension/lizzy/plugin/parts/DisplayView.java 0% <0%> (ø) ⬆️
...va/de/intension/lizzy/plugin/parts/SearchView.java 0% <0%> (ø) ⬆️
...tension/lizzy/converter/gherkin/GherkinParser.java 60% <0%> (-15%) ⬇️
...y/plugin/dialogs/ConverterConfigurationDialog.java 0% <0%> (ø) ⬆️
...sion/lizzy/converter/gherkin/GherkinGenerator.java 81.25% <50%> (-3.6%) ⬇️
... and 1 more

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 69ca3ca...40cb92d. Read the comment docs.

@ingokuba ingokuba force-pushed the feature/#5/addSonar branch 6 times, most recently from 58bb7f8 to aef2340 Compare August 30, 2018 14:16
@ingokuba ingokuba added the help wanted Extra attention is needed label Aug 30, 2018
@ingokuba ingokuba removed the request for review from rbauer-int August 30, 2018 15:12
@ingokuba ingokuba force-pushed the feature/#5/addSonar branch 2 times, most recently from a4b48aa to ccea926 Compare August 30, 2018 15:47
@ingokuba ingokuba added this to In progress in Lizzy board Aug 31, 2018
@ingokuba ingokuba removed the help wanted Extra attention is needed label Aug 31, 2018
@ingokuba ingokuba moved this from In progress to To do in Lizzy board Sep 13, 2018
@ingokuba ingokuba force-pushed the feature/#5/addSonar branch 4 times, most recently from 61954cf to 08e5305 Compare October 2, 2018 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Lizzy board
  
To do
Development

Successfully merging this pull request may close these issues.

None yet

1 participant