Skip to content
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

Restructure for Arduino Library #26

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

16aroth6
Copy link

Restructured files for compliance with the Arduino Library Specification found here:
https://arduino.github.io/arduino-cli/1.0/library-specification/

Changes consisted of:
Moving source files to the src/ directory.
Adding the library.properties file for proper Arduino Library management.
Adding ArTICL source files to match the library naming convention and for future development.
Adding a generic keywords.txt document that is used by the Arduino IDE and web app to highlight specific types.

As edited, when placed in the Arduino IDE libraries folder like the manager would do, it is recognized and the examples are available from the menu.
image

@16aroth6
Copy link
Author

I missed a typo in library.properties, the sentance variable needs to be sentence

@16aroth6
Copy link
Author

Passes Arduino Lint with no errors.
image

Repository owner deleted a comment from DoctorWho8 Aug 10, 2024
Copy link
Owner

@KermMartian KermMartian left a comment

Choose a reason for hiding this comment

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

A few minor change requests.

library.properties Outdated Show resolved Hide resolved
src/ArTICL.h Outdated Show resolved Hide resolved
library.properties Outdated Show resolved Hide resolved
Co-authored-by: Christopher Mitchell, Ph.D. <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants