Skip to content

Conversation

MatteoPologruto
Copy link
Contributor

@MatteoPologruto MatteoPologruto commented Feb 22, 2023

Please check if the PR fulfills these requirements

See how to contribute

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • The PR follows
    our contributing guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • 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 as is_legacy: true libraries.

What is the new behavior?

Folders that do not contain a library.properties or a .h file are ignored when lib list is executed.

Does this PR introduce a breaking change, and is titled accordingly?

No

@MatteoPologruto MatteoPologruto added topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project labels Feb 22, 2023
@MatteoPologruto MatteoPologruto self-assigned this Feb 22, 2023
@codecov
Copy link

codecov bot commented Feb 22, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.11 🎉

Comparison is base (8c2b8e7) 62.44% compared to head (c3adae2) 62.56%.

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     
Flag Coverage Δ
unit 62.56% <100.00%> (+0.11%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
arduino/libraries/loader.go 82.66% <100.00%> (+2.21%) ⬆️

... 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.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Contributor

@umbynos umbynos left a 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

@MatteoPologruto MatteoPologruto force-pushed the lib-list-dot-h branch 2 times, most recently from e526800 to 37df1d4 Compare March 22, 2023 14:32
Copy link
Member

@cmaglie cmaglie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very good!

@MatteoPologruto MatteoPologruto merged commit b69ed33 into arduino:master Mar 28, 2023
@MatteoPologruto MatteoPologruto deleted the lib-list-dot-h branch March 28, 2023 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

lib list should not report empty folders under <directories.user>/libraries as a is_legacy library
3 participants