Skip to content

Removes kinetic from travis ci testing#44

Merged
rhaschke merged 2 commits intomoveit:masterfrom
PilzDE:remove_kinetic_ci
Sep 28, 2020
Merged

Removes kinetic from travis ci testing#44
rhaschke merged 2 commits intomoveit:masterfrom
PilzDE:remove_kinetic_ci

Conversation

@rfeistenauer
Copy link
Copy Markdown
Contributor

According to the removal of the kinetic CI in ros-planning/moveit #2294 the CI can/should? be removed from this repository as well.
It is already causing troubles see #43

Copy link
Copy Markdown
Contributor

@rhaschke rhaschke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Current master branch of moveit_resources works just fine on Kinetic:
https://travis-ci.com/github/ros-planning/moveit_resources/builds/180393960

#43, introducing an IKFastPlugin, newly makes it compatible with Kinetic.
However, as we will not release versions > 0.7 into Kinetic, I'm fine to drop Kinetic support on Travis.

@rhaschke
Copy link
Copy Markdown
Contributor

Instead of Kinetic testing, we should add Noetic.

@jschleicher
Copy link
Copy Markdown
Contributor

@henningkayser could you merge this please?

@ct2034 ct2034 mentioned this pull request Sep 28, 2020
13 tasks
@ct2034
Copy link
Copy Markdown
Contributor

ct2034 commented Sep 28, 2020

@rhaschke Is there something else blocking this at the moment?

@rhaschke
Copy link
Copy Markdown
Contributor

Formally, I was waiting for a 2nd approval. However, I'm going to merge this now.

@rhaschke rhaschke merged commit d72c9ad into moveit:master Sep 28, 2020
@jschleicher jschleicher deleted the remove_kinetic_ci branch September 28, 2020 15:32
@ct2034
Copy link
Copy Markdown
Contributor

ct2034 commented Sep 28, 2020

thanks Robert

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.

4 participants