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

Apply path filters based on relative paths when possible #6330

Merged
merged 2 commits into from
Aug 3, 2023

Conversation

3flex
Copy link
Member

@3flex 3flex commented Jul 30, 2023

Stacks on #6323

Fixes #3728

@codecov
Copy link

codecov bot commented Jul 30, 2023

Codecov Report

Merging #6330 (e99fcec) into main (51e6065) will increase coverage by 0.02%.
Report is 5 commits behind head on main.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main    #6330      +/-   ##
============================================
+ Coverage     85.16%   85.18%   +0.02%     
+ Complexity     4029     4028       -1     
============================================
  Files           571      571              
  Lines         13420    13420              
  Branches       2369     2366       -3     
============================================
+ Hits          11429    11432       +3     
  Misses          833      833              
+ Partials       1158     1155       -3     
Files Changed Coverage Δ
...tlab/arturbosch/detekt/api/internal/PathFilters.kt 96.77% <100.00%> (+0.22%) ⬆️
.../kotlin/io/gitlab/arturbosch/detekt/cli/CliArgs.kt 100.00% <100.00%> (ø)
...io/gitlab/arturbosch/detekt/core/KtTreeCompiler.kt 72.72% <100.00%> (ø)
...etekt/core/tooling/ProcessingSpecSettingsBridge.kt 100.00% <100.00%> (+6.66%) ⬆️
.../main/kotlin/io/github/detekt/parser/KtCompiler.kt 87.50% <100.00%> (+6.73%) ⬆️
...io/github/detekt/tooling/dsl/ProjectSpecBuilder.kt 100.00% <100.00%> (ø)

@cortinico cortinico added this to the 2.0.0 milestone Jul 30, 2023
@3flex 3flex added the notable changes Marker for notable changes in the changelog label Aug 3, 2023
@3flex 3flex merged commit 67cd931 into detekt:main Aug 3, 2023
23 checks passed
@BraisGabin BraisGabin mentioned this pull request Jan 8, 2024
mgroth0 pushed a commit to mgroth0/detekt that referenced this pull request Feb 11, 2024
* Require basePath when setting up environment

This is a prerequisite for fixing detekt#3728

* Apply path filters based on relative paths when possible
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api cli core notable changes Marker for notable changes in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Baseline incomplete if project located in "test" directory.
3 participants