-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
Multiple dirs class set #400
base: main
Are you sure you want to change the base?
Conversation
How about refactoring The constructor is |
Yes! That was my first approach, but I found some reason to do it differently... Dont remember what it was though. :( We can even change the named constructor without a BC break. my only concern here is with the Anyway, as I can't see anymore what drove me to this approach, I will simplify as you suggest. |
a9b0366
to
501feda
Compare
@jdreesen done, and force pushed |
501feda
to
e1a979b
Compare
e1a979b
to
f27b65a
Compare
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. @@ Coverage Diff @@
## main #400 +/- ##
=========================================
Coverage 94.33% 94.33%
Complexity 571 571
=========================================
Files 67 67
Lines 1500 1500
=========================================
Hits 1415 1415
Misses 85 85
|
@hgraca Before approving, can you please add something to the readme file :) ? |
f27b65a
to
990416a
Compare
This way we can specify just the directories we want it to scan, as opposed to include all dirs in the root and then excluding all dirs we don't want in the set.
990416a
to
7aa145e
Compare
Currently, if we want to include two folders from the project root, we need to add the root of the project and then exclude everything we dont want to include, and remember to add a folder to the config if we add a new folder to the project root.
With the
MultipleDirClassSet
we can specify just the directories we want it to scan,removing all cruft from our config.