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

Remove line separator detection logic #7251

Merged
merged 1 commit into from May 6, 2024
Merged

Conversation

3flex
Copy link
Member

@3flex 3flex commented May 3, 2024

This is now fully handled by the IntelliJ components in the embeddable compiler.

Copy link

codecov bot commented May 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.74%. Comparing base (60b2413) to head (3eb8592).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #7251      +/-   ##
============================================
- Coverage     84.75%   84.74%   -0.01%     
  Complexity     3996     3996              
============================================
  Files           577      577              
  Lines         12133    12126       -7     
  Branches       2486     2483       -3     
============================================
- Hits          10283    10276       -7     
  Misses          624      624              
  Partials       1226     1226              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cortinico
Copy link
Member

@3flex is this ready for review?

@BraisGabin BraisGabin marked this pull request as ready for review May 6, 2024 17:42
@BraisGabin
Copy link
Member

I messed up from mobile and I can move it back to draft

@BraisGabin BraisGabin marked this pull request as draft May 6, 2024 17:46
@3flex 3flex marked this pull request as ready for review May 6, 2024 21:12
This is now full handled by the IntelliJ components in the embeddable
compiler.
@3flex 3flex enabled auto-merge (squash) May 6, 2024 22:21
@3flex 3flex merged commit 0951519 into detekt:main May 6, 2024
21 checks passed
@3flex 3flex deleted the location-7 branch May 6, 2024 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants