-
Notifications
You must be signed in to change notification settings - Fork 3
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 Automated Build/Tests #14
Comments
agreed, thank you for the feedback! |
I will try to find time to work on this soon. Any chance you could help out here a little @cmower? Or are you too occupied? No worries if not. |
automatic builds are already added for Ubuntu 24.04 and 22.04. Currently missing are tests |
I think automated builds alone should be sufficient as this repository primarily contains bindings. |
thanks for the feedback @CameronDevine, I am in the process of writing an emulator for the robot to mimic its behavior. This should allow for simple testing by the end of the week. |
maybe in the interest of time, is it possible to treat the procedural tests with virtual hardware separately from this review? I would then open new issues and reference this but keep JOSS untagged. The progress of the emulator can be tracked here : https://github.com/lbr-stack/fri_emulator |
I don't think closing this issue is required for me to recommend acceptance. @adi3 is this correct? |
As part of the JOSS review (openjournals/joss-reviews#6138) I think this repository should have an automated build, preferably with some tests.
The text was updated successfully, but these errors were encountered: