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

Updated c++ version from c++11 to c++14 #387

Merged
merged 1 commit into from
Nov 21, 2019

Conversation

JonasTietz
Copy link
Contributor

Description

There was conflicting information about the c++ version used.
Now all c++ versions are c++14.

@welcome
Copy link

welcome bot commented Nov 21, 2019

Thanks for helping in improving MoveIt!

@rhaschke rhaschke merged commit e4e9344 into moveit:gh-pages Nov 21, 2019
@welcome
Copy link

welcome bot commented Nov 21, 2019

Congrats on getting your first MoveIt! pull request merged and improving open source robotics!

@NFiedler NFiedler mentioned this pull request Nov 21, 2019
@davetcoleman
Copy link
Member

davetcoleman commented Nov 21, 2019

Yesterday when updating this page, I noticed this inconsistency also. I had added this to the agenda for the next MoveIt maintainer meeting - I don't think we ever officially decided to switch to C++14 did we? @rhaschke ?

@davetcoleman
Copy link
Member

Wait it looks like we did, actually, 1 year ago :-)
https://github.com/ros-planning/moveit/pull/1146/files

@rhaschke
Copy link
Contributor

Yes, we did.

davetcoleman pushed a commit to davetcoleman/moveit.ros.org that referenced this pull request Aug 19, 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.

4 participants