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

Add continuous integration #453

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

Conversation

Gadgetoid
Copy link
Collaborator

Adds a very basic build of the library using CMake to catch elementary errors.

Very interested in suggestions for how this could be improved- albeit full unit tests and such are probably a folly at this stage.

@Gadgetoid
Copy link
Collaborator Author

@jgarff should probably also mention this will run against your GitHub Actions quota, I believe, so you might want to yay/nay this.

@Macleykun
Copy link

Just wondering
But is jgarff still active?
And if not, this pr could be merged? I assume it works on your fork :) ?

@Gadgetoid Gadgetoid force-pushed the continuous-integration branch from 66a168c to 8190fe5 Compare March 6, 2023 14:42
@Gadgetoid
Copy link
Collaborator Author

I still have reservations about forcing potentially billable activities upon someone, but even if they're currently fine. I'd definitely like a yay or nay from the source, but I'll rebase to keep this from going totally defunct.

I... can't say that I'm super active at the moment. The Pi shortage really forced a priority shift so my attention has been elsewhere 😬

@jgarff are you out there?

@jgarff
Copy link
Owner

jgarff commented Oct 19, 2024

Sorry guys, I completely missed your comments here. Sorry about that.

@Gadgetoid
Copy link
Collaborator Author

Sorry guys, I completely missed your comments here. Sorry about that.

Story of my life.

As to whether this still works, and is (or ever was) the right approach... I need to find the time to check!

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.

3 participants