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

PlatformIO layout #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mumrah
Copy link

@mumrah mumrah commented Nov 6, 2017

This PR reorganizes the source files to make the repo compatible with PlatformIO. It also creates a Arduino sketch with the same name as the directory which is a requirement of the Arduino IDE.

With thing change I can:

  • Open "bitx40.ino" with Arduino IDE and compile/upload the Raduino (just like today). No need to "copy the sketch into a folder of the same name"
  • Open as a PlatformIO project and compile/upload to the Raduino

What this allows:

  • Development in PlatformIO (many nice things about this compared to Arduino IDE)
  • Eventual refactoring of large sketch into smaller private libraries
  • Easier writing of unit tests and integration with TravisCI

At first, I tried pulling the Si5351 code into a project-local library (which is supported by PlatformIO), but couldn't get it working via the INO sketch. Something to do with #include and my lacking C skills.

Is this something people would be interested in?

@davehun
Copy link

davehun commented Feb 7, 2018

great suggestion I was just about to do the same thing!

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.

None yet

2 participants