-
Notifications
You must be signed in to change notification settings - Fork 2
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
Setup CI #7
Setup CI #7
Conversation
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Had retriggered checks, but its only failing for copyright |
Thanks for the review, @quarkytale ! Now that #2 and #5 have been merged, I expect the linters to fail. Once CI comes back, if only the linters are failing, I'll merge this PR, and you can work against |
Ok, only linter errors:
Merging! |
Add GitHub actions-based CI that runs on every push and PR.
It uses binaries for ROS and Gazebo dependencies, pulled from https://packages.ros.org, and compiles from source all the repos in buoy_all.yaml.
Then it tests all buoy repos, including
buoy_msgs
. I think we should just rely on the ROS buildfarm to run CI for the messages package eventually, but until we're ready to bloom it, we can test it here.CI is currently failing the copyright check, which should be fixed by #6