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

Remove melodic ci #147

Merged
merged 1 commit into from
May 22, 2024
Merged

Remove melodic ci #147

merged 1 commit into from
May 22, 2024

Conversation

snosenzo
Copy link
Contributor

The ros CI action to use node v20 which can no longer run on this version of ubuntu.

@defunctzombie
Copy link
Contributor

Is melodic making anymore bloom releases? If it is not then there's little value in continuing to test it and make builds for it.

@achim-k where would we find this information?

@achim-k
Copy link
Collaborator

achim-k commented May 22, 2024

It's EOL since about a year (https://wiki.ros.org/Distributions). So you can't make new releases for it.

@jtbandes jtbandes merged commit 17b2332 into update-annotation-schemas May 22, 2024
8 of 9 checks passed
@jtbandes jtbandes deleted the remove-melodic-ci branch May 22, 2024 23:53
@snosenzo
Copy link
Contributor Author

🤦 This wasn't off of main it was off of the other PR. Gonna re-do this

jtbandes pushed a commit that referenced this pull request May 23, 2024
The ros CI action to use node v20 which can no longer run on this
version of ubuntu.

This is a repeat of #147 because that was just merged into another
branch. Sorry for the confusion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants