-
Notifications
You must be signed in to change notification settings - Fork 3
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
Implement EXCLUSIONS config value #120
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #120 +/- ##
=======================================
Coverage ? 22.57%
=======================================
Files ? 13
Lines ? 753
Branches ? 145
=======================================
Hits ? 170
Misses ? 578
Partials ? 5 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The built in list feature class filtering is for including not excluding right? If that's the case this solution seems great.
Yes, the |
Without this change, breakpoints did not work correctly.
Such as 20th, 3rd, or 2nd. Closes #57
I wasn't sure if your last comment indicated that you had reviewed the code yet. So I added some work that I did this morning to this PR. |
No description provided.