-
-
Notifications
You must be signed in to change notification settings - Fork 794
Refactoring: use more readable functions #1916
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1916 +/- ##
========================================
Coverage 80.4% 80.4%
+ Complexity 1974 1973 -1
========================================
Files 326 326
Lines 5517 5517
Branches 1027 1027
========================================
Hits 4436 4436
Misses 543 543
Partials 538 538
Continue to review full report at Codecov.
|
@@ -21,7 +21,7 @@ class KotlinEnvironmentUtilsTest : Spek({ | |||
.toList() | |||
|
|||
val languageVersion = classpathFiles.getKotlinLanguageVersion() | |||
assertThat(languageVersion).isNotNull() | |||
assertThat(languageVersion).isNotNull |
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.
Interesting, on which IntelliJ version are you and is the new kotlin type inference enabled in kontlinc by default in 1.3.50? Last time I removed braces for assertJ "getters" I got compiler errors in IDEA.
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.
2018.3
No description provided.