-
Notifications
You must be signed in to change notification settings - Fork 8
[CI] Reusable CI #24
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
[CI] Reusable CI #24
Conversation
|
The checks are really failing but probably not set to be needed to merge? |
|
Not sure if it ever worked or if it's failing with the same error now as 3 months ago: |
|
Looks like it's just style complaints: |
Signed-off-by: Esteban Martinena <[email protected]>
531dd70 to
94a50fe
Compare
Fixed those but it looks like for focal ones it's the same problem we found here: open-rmf/rmf#202 (comment) We could try to go with any of the two options suggested here and wait (open-rmf/rmf#202 (comment)) or I could avoid the workflow to run in focal-based distros. What do you think? |
Signed-off-by: Esteban Martinena <[email protected]>
39ddcea to
26f642a
Compare
Signed-off-by: Esteban Martinena <[email protected]>
Signed-off-by: Esteban Martinena <[email protected]>
Signed-off-by: Esteban Martinena <[email protected]>
Signed-off-by: Esteban Martinena <[email protected]>
26f642a to
5dcd92d
Compare
Modified build.yaml to use reusable workflow.
Signed-off-by: Esteban Martinena [email protected]