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
Fix SpacingBetweenPackageAndImports issue for scripts without packages #1947
Conversation
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.
review: @sschuberth
...s/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/SpacingBetweenPackageAndImports.kt
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #1947 +/- ##
============================================
- Coverage 80.75% 80.74% -0.02%
+ Complexity 2001 2000 -1
============================================
Files 329 329
Lines 5587 5587
Branches 1038 1038
============================================
- Hits 4512 4511 -1
Misses 537 537
- Partials 538 539 +1
Continue to review full report at Codecov.
|
This rule doesn't use a mutable variable anymore.
detekt#1947) * Revert "Fix SpacingBetweenPackageAndImports issue for KTS (detekt#1939)" This reverts commit faff9f4. * Fix SpacingBetweenPackageAndImports issue for scripts without packages Both a Kotlin file and a Kotlin script can be declared with and without a package declaration. This commit checks if the class even contains a package declaration for both .kt and .kts files. Closes detekt#1937 * Use more sophisticated package check * Refactor SpacingBetweenPackageAndImports rule This rule doesn't use a mutable variable anymore.
detekt#1947) * Revert "Fix SpacingBetweenPackageAndImports issue for KTS (detekt#1939)" This reverts commit faff9f4. * Fix SpacingBetweenPackageAndImports issue for scripts without packages Both a Kotlin file and a Kotlin script can be declared with and without a package declaration. This commit checks if the class even contains a package declaration for both .kt and .kts files. Closes detekt#1937 * Use more sophisticated package check * Refactor SpacingBetweenPackageAndImports rule This rule doesn't use a mutable variable anymore.
detekt#1947) * Revert "Fix SpacingBetweenPackageAndImports issue for KTS (detekt#1939)" This reverts commit faff9f4. * Fix SpacingBetweenPackageAndImports issue for scripts without packages Both a Kotlin file and a Kotlin script can be declared with and without a package declaration. This commit checks if the class even contains a package declaration for both .kt and .kts files. Closes detekt#1937 * Use more sophisticated package check * Refactor SpacingBetweenPackageAndImports rule This rule doesn't use a mutable variable anymore.
Both a Kotlin file and a Kotlin script can be declared with and without
a package declaration. This commit checks if the class even contains a
package declaration for both .kt and .kts files.
Closes #1937