Skip to content

Added Qodana support (experiment) #703

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

Merged
merged 2 commits into from
Feb 22, 2023
Merged

Added Qodana support (experiment) #703

merged 2 commits into from
Feb 22, 2023

Conversation

bodiam
Copy link
Contributor

@bodiam bodiam commented Feb 12, 2023

Qodana is a code quality platform similar to Snyk. This is an experiment to try it out. See for more info https://www.youtube.com/watch?v=WrhnUnzMUCg&t=203s or https://www.jetbrains.com/qodana/

@codecov-commenter
Copy link

Codecov Report

Merging #703 (0088e8b) into main (fd0ab2c) will decrease coverage by 0.02%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##               main     #703      +/-   ##
============================================
- Coverage     92.69%   92.67%   -0.02%     
+ Complexity     2581     2580       -1     
============================================
  Files           281      281              
  Lines          5121     5121              
  Branches        532      532              
============================================
- Hits           4747     4746       -1     
  Misses          240      240              
- Partials        134      135       +1     
Impacted Files Coverage Δ
...ain/java/net/datafaker/providers/base/Barcode.java 85.71% <0.00%> (-4.77%) ⬇️
.../java/net/datafaker/service/FakeValuesService.java 84.28% <0.00%> (-0.20%) ⬇️
...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java 96.29% <0.00%> (+3.70%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@kingthorin
Copy link
Collaborator

kingthorin commented Feb 16, 2023

Looks like docker has to be installed on the runners to use this.

@bodiam
Copy link
Contributor Author

bodiam commented Feb 16, 2023

Yes, seems like it. I'm probably using the wrong build file, need to look into this. If you have an easy fix, please do share :)

@bodiam bodiam merged commit 413d090 into main Feb 22, 2023
@kingthorin kingthorin deleted the qodana_support branch February 22, 2023 11:27
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