-
-
Notifications
You must be signed in to change notification settings - Fork 398
[skip-changelog] Exclude libraries with a missing .h
file from lib list
#2083
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
[skip-changelog] Exclude libraries with a missing .h
file from lib list
#2083
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #2083 +/- ##
==========================================
+ Coverage 62.44% 62.56% +0.11%
==========================================
Files 231 231
Lines 19593 19610 +17
==========================================
+ Hits 12235 12269 +34
+ Misses 6254 6238 -16
+ Partials 1104 1103 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 3 files with indirect coverage changes 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 in Codecov by Sentry. |
14b5c8e
to
405e0e1
Compare
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.
We should also update the docs in this PR
e526800
to
37df1d4
Compare
bb30f0a
to
b880ec4
Compare
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.
Very good!
b880ec4
to
931c328
Compare
3732ae3
to
c3adae2
Compare
Please check if the PR fulfills these requirements
See how to contribute
before creating one)
our contributing guidelines
UPGRADING.md
has been updated with a migration guide (for breaking changes)What kind of change does this PR introduce?
Code imperfection fix
What is the current behavior?
Empty folders under
<directories.user>/libraries
show up asis_legacy: true
libraries.What is the new behavior?
Folders that do not contain a
library.properties
or a.h
file are ignored whenlib list
is executed.Does this PR introduce a breaking change, and is titled accordingly?
No