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

Improve dependencies management #1340

Merged
merged 7 commits into from Dec 3, 2020

Conversation

yuri-sergiichuk
Copy link
Contributor

This PR improves our Gradle dependencies management a bit and applies the new scope to the dependencies that are used by the code analysis software only.

The change is related to the SpineEventEngine/config#156 issue.

@yuri-sergiichuk yuri-sergiichuk self-assigned this Dec 3, 2020
@codecov
Copy link

codecov bot commented Dec 3, 2020

Codecov Report

Merging #1340 (85f7ef4) into master (c5e8a92) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1340   +/-   ##
=========================================
  Coverage     91.03%   91.03%           
  Complexity     4774     4774           
=========================================
  Files           614      614           
  Lines         15146    15146           
  Branches        860      860           
=========================================
  Hits          13788    13788           
  Misses         1088     1088           
  Partials        270      270           

@yuri-sergiichuk yuri-sergiichuk requested a review from a team December 3, 2020 15:32
Copy link
Contributor

@alexander-yevsyukov alexander-yevsyukov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yuri-sergiichuk yuri-sergiichuk merged commit 4dabb11 into master Dec 3, 2020
@yuri-sergiichuk yuri-sergiichuk deleted the improve-dependencies-management branch December 3, 2020 15:47
@yuri-sergiichuk yuri-sergiichuk mentioned this pull request Dec 14, 2020
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.

None yet

2 participants