-
Notifications
You must be signed in to change notification settings - Fork 39
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
Fixes in code style, update documentation #930
Conversation
### What's done: * Added rule 3.16.2 verbose description * Fixed typos in rule 6.2.2
### What's done: * Updated docs * Fixed and updated gradle scripts for docs generation
Codecov Report
@@ Coverage Diff @@
## master #930 +/- ##
=========================================
Coverage 84.15% 84.15%
Complexity 2374 2374
=========================================
Files 101 101
Lines 6008 6008
Branches 1773 1773
=========================================
Hits 5056 5056
Misses 263 263
Partials 689 689
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@@ -857,13 +863,14 @@ If the classes are meant to be used externally, and are not referenced inside th | |||
- Companion object | |||
|
|||
**Exception:** | |||
All variants of a `(private) val` logger should be placed at the beginning of the class (`(private) val log`, `LOG`, `logger`, etc.). | |||
All variants of a `private val` logger should be placed at the beginning of the class (`private val log`, `LOG`, `logger`, etc.). |
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.
have you generated it?
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.
yes, why?
What's done:
This pull request closes #928