Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

skip excluded_folders when getting cached filepaths from root #210

Merged
merged 1 commit into from
May 16, 2023
Merged

skip excluded_folders when getting cached filepaths from root #210

merged 1 commit into from
May 16, 2023

Conversation

mike-hoang
Copy link
Contributor

@mike-hoang mike-hoang commented May 16, 2023 •
edited
Loading

What does this PR do?

Uses the excluded_folders from languages-customization.yml to skip go vendor directories when getting the cached filepaths.

➜  alizer-testing time ./alizer-old component eventing-kafka-broker > old.log
./alizer-old component eventing-kafka-broker > old.log  5.91s user 0.87s system 104% cpu 6.514 total

➜  alizer-testing time ./alizer component eventing-kafka-broker > new.log
./alizer component eventing-kafka-broker > new.log  3.20s user 0.31s system 102% cpu 3.443 total

Which issue(s) this PR fixes:

fixes #208

@mike-hoang
skip excluded_folders when getting cached filepaths from root
68df2b5 
Signed-off-by: Michael Hoang <mhoang@redhat.com>
@mike-hoang mike-hoang requested a review from yangcao77 May 16, 2023 18:47
@mike-hoang mike-hoang marked this pull request as ready for review May 16, 2023 18:47
@thepetk thepetk self-requested a review May 16, 2023 21:45
@mike-hoang mike-hoang merged commit 135a2bb into redhat-developer:main May 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@thepetk thepetk thepetk approved these changes

@yangcao77 yangcao77 Awaiting requested review from yangcao77

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Alizer component scanning should exclude vendor folders
2 participants
@mike-hoang @thepetk