-
Notifications
You must be signed in to change notification settings - Fork 39
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
Check for filetype only for pattern and fix coverage #872
Conversation
Codecov ReportBase: 93.12% // Head: 93.17% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #872 +/- ##
==========================================
+ Coverage 93.12% 93.17% +0.05%
==========================================
Files 47 47
Lines 2095 2096 +1
Branches 268 268
==========================================
+ Hits 1951 1953 +2
Misses 109 109
+ Partials 35 34 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
42e98b1
to
0038832
Compare
Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
for more information, see https://pre-commit.ci
Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
for more information, see https://pre-commit.ci
# Description ## What is the current behavior? <!-- Please describe the current behavior that you are modifying. --> Currently is filetype is passed then pattern doesn't need to be resolved to check native autodetect <!-- Issues are required for both bug fixes and features. Reference it using one of the following: closes: #ISSUE related: #ISSUE --> closes: #873 ## What is the new behavior? <!-- Please describe the behavior or changes that are being added by this PR. --> - Create file type in case not provided before checking for native autodetect schema ## Does this introduce a breaking change? No ### Checklist - [x] Created tests which fail without the change (if possible) - [x] Extended the README / documentation, if necessary Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
# Description ## What is the current behavior? <!-- Please describe the current behavior that you are modifying. --> Currently is filetype is passed then pattern doesn't need to be resolved to check native autodetect <!-- Issues are required for both bug fixes and features. Reference it using one of the following: closes: #ISSUE related: #ISSUE --> closes: #873 ## What is the new behavior? <!-- Please describe the behavior or changes that are being added by this PR. --> - Create file type in case not provided before checking for native autodetect schema ## Does this introduce a breaking change? No ### Checklist - [x] Created tests which fail without the change (if possible) - [x] Extended the README / documentation, if necessary Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
# Description ## What is the current behavior? <!-- Please describe the current behavior that you are modifying. --> Currently is filetype is passed then pattern doesn't need to be resolved to check native autodetect <!-- Issues are required for both bug fixes and features. Reference it using one of the following: closes: #ISSUE related: #ISSUE --> closes: #873 ## What is the new behavior? <!-- Please describe the behavior or changes that are being added by this PR. --> - Create file type in case not provided before checking for native autodetect schema ## Does this introduce a breaking change? No ### Checklist - [x] Created tests which fail without the change (if possible) - [x] Extended the README / documentation, if necessary Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
Description
What is the current behavior?
Currently is filetype is passed then pattern doesn't need to be resolved to check native autodetect
closes: #873
What is the new behavior?
Does this introduce a breaking change?
No
Checklist