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
fix: Remove Unneeded/Redundant Manifest Fields #58
fix: Remove Unneeded/Redundant Manifest Fields #58
Conversation
misc: merge from upstream
- list ArduinoJson as a dependency. - add Arduino sketch path to config. - fix the 'platformio_example' path.
This reverts commit 55d830b.
- Releases should include all files from the 'root/' dir for the source code zip. - per PIO, the "examples field" not being present defaults to the paths that are already shown. So this part is just redundant.
@air1one @faustbrian - please review this in the next few days. Be sure to explicitly select labels so I know what's going on. If no reviewer appears after a week, a reminder will be sent out. |
@sleepdefic1t The ci/circleci: build-linux-clang-5 job is failing as of 4fbf5681d102112f389373ffa1c55e8a675714c7. Please review the logs for more information. Once you've pushed the fixes, the build will automatically re-run. Thanks! |
Failing test is known in #7, the changes in this PR do not affect this failure. |
Proposed changes
This PR removes the following fields from
library.json
:Releases should include all files and folders from the
root/
dir for the source code zip.Also, Per PIO, the "examples" field not being present defaults to the paths that are already shown. So this part is just redundant.
Types of changes
Checklist