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

Update ROS 2 distributions #199

Merged
merged 3 commits into from
Jan 4, 2020
Merged

Update ROS 2 distributions #199

merged 3 commits into from
Jan 4, 2020

Conversation

nuclearsandwich
Copy link

Bouncy and Crystal have both reached end of support and Eloquent hasn't been added yet.

cc #123

@nuclearsandwich nuclearsandwich self-assigned this Dec 14, 2019
@dirk-thomas
Copy link
Member

Instead of maintaining a separate list in this repo it would be better to use the information available from rosdistro.

@nuclearsandwich
Copy link
Author

Instead of maintaining a separate list in this repo it would be better to use the information available from rosdistro.

Which is why #123 is open and mentioned above.

@dirk-thomas
Copy link
Member

Which is why #123 is open and mentioned above.

But still the data is being updated separately by this PR. I just wanted to point out that it might be better to just consolidate it now than keep updating it separately yet another time.

@tfoote tfoote merged commit 6dc8c1d into ros2 Jan 4, 2020
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