-
-
Notifications
You must be signed in to change notification settings - Fork 757
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 documentation for Gradle repositories setup #3030
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3030 +/- ##
=========================================
Coverage 79.31% 79.31%
Complexity 2562 2562
=========================================
Files 432 432
Lines 7725 7725
Branches 1466 1466
=========================================
Hits 6127 6127
Misses 814 814
Partials 784 784 Continue to review full report at Codecov.
|
We need to do something with |
content { | ||
// just allow to include kotlinx projects | ||
// detekt needs 'kotlinx-html' for the html report | ||
includeGroup "org.jetbrains.kotlinx" |
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.
hm, I would really like to leave one of these for reference on.
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.
Agree. Maybe we can refactor this into a ## Repositories
section and have it listed only once?
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.
I implemented the @cortinico suggestion. What do you think?
0f73db6
to
d15a1be
Compare
Right now our gradle documentation is really verbose It repeats the same block of code 5 times so it's difficult to find what you are looking for.
I do think that jcenter is not the best repository and qw should be avoid it when possible. But that's the user choice and our documentation is not the place where we should promote that.