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 PsiFile.basePath #7237

Merged
merged 6 commits into from May 5, 2024
Merged

Remove PsiFile.basePath #7237

merged 6 commits into from May 5, 2024

Conversation

3flex
Copy link
Member

@3flex 3flex commented Apr 28, 2024

No description provided.

Copy link

codecov bot commented May 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.78%. Comparing base (65f3164) to head (cb3d8f5).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #7237   +/-   ##
=========================================
  Coverage     84.77%   84.78%           
- Complexity     3993     3994    +1     
=========================================
  Files           578      578           
  Lines         12138    12136    -2     
  Branches       2484     2483    -1     
=========================================
- Hits          10290    10289    -1     
+ Misses          624      623    -1     
  Partials       1224     1224           

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

@detekt-ci detekt-ci added the cli label May 5, 2024
@BraisGabin
Copy link
Member

BraisGabin commented May 5, 2024

Probably we should forbid any usage of absolute() or similar functions called from our core or reporters.

It's not doable right now but it would prevent issues in the future. I'll add that as soon as I finish with option to instantiate the same rule more than once.

@3flex 3flex merged commit 1c12968 into detekt:main May 5, 2024
21 checks passed
@3flex 3flex deleted the location-3 branch May 5, 2024 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api breaking change Marker for breaking changes which should be highlighted in the changelog cli compiler-plugin core parser psi-utils
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants