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

Simplify UnnecessaryApplySpec test cases #2448

Merged
merged 4 commits into from Mar 18, 2020
Merged

Simplify UnnecessaryApplySpec test cases #2448

merged 4 commits into from Mar 18, 2020

Conversation

schalkms
Copy link
Member

  • Fixes invalid Kotlin code in test snippets
  • Use compileAndLint() instead of lint()

* Fixes invalid Kotlin code in test snippets
* Use compileAndLint() instead of lint()
* Uses safe-cast expression
* Doesn't use unsupported inline classes
@codecov-io
Copy link

Codecov Report

Merging #2448 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2448   +/-   ##
=========================================
  Coverage     82.94%   82.94%           
  Complexity     2178     2178           
=========================================
  Files           359      359           
  Lines          6226     6226           
  Branches       1143     1143           
=========================================
  Hits           5164     5164           
  Misses          480      480           
  Partials        582      582

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db87ebb...a0645c6. Read the comment docs.

Co-Authored-By: Brais Gabín <braisgabin@gmail.com>
@schalkms schalkms merged commit 7f5ce20 into detekt:master Mar 18, 2020
@schalkms schalkms deleted the simplify-UnnecessaryApplySpec branch March 18, 2020 19:30
@arturbosch arturbosch added this to the 1.7.0 milestone Mar 19, 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

4 participants