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.

Sign up for GitHub

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

Jump to bottom

Switch to clang-format-6 or *-10 #235

Closed
henningkayser opened this issue Jul 23, 2020 · 2 comments
Closed

Switch to clang-format-6 or *-10 #235

henningkayser opened this issue Jul 23, 2020 · 2 comments

Comments

@henningkayser
Copy link
Member

henningkayser commented Jul 23, 2020 •
edited
Loading

With Foxy/Noetic (Ubuntu Focal) clang-format-3.9 is not supported anymore. In order to allow syncing between MoveIt 1 and MoveIt 2 we need to find a common clang-format version can be applied for all distros. For now, I suggest running clang-format-3.9 on Eloquent (see 7aea7a7 from #227) until we found a good replacement.

Related:
@henningkayser
Copy link
Member Author

moveit/moveit#2177 has been merged, I will perform a new repo sync and then file a separate PR fixing clang-format-10.

@JafarAbdi
Copy link
Member

Fixed in #282

MikeWrock pushed a commit to MikeWrock/moveit2 that referenced this issue Aug 15, 2022
@DLu
New Doc Link (moveit#235)
0c69b1c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@henningkayser @JafarAbdi