Skip to content
This repository has been archived by the owner on Dec 10, 2020. It is now read-only.

Create new repo uncrutify_vendor and deprecate this one #24

Closed
mikaelarguedas opened this issue Jun 27, 2018 · 4 comments
Closed

Create new repo uncrutify_vendor and deprecate this one #24

mikaelarguedas opened this issue Jun 27, 2018 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@mikaelarguedas
Copy link

Follow up of #23

Now that recent uncrustify versions have been released in debian we should use upstream where we can and build uncrustify only on platforms that need it

@mikaelarguedas
Copy link
Author

mikaelarguedas commented Jul 9, 2018

New round of CI for all platforms including latest changes:

This won't be merged before the overall branching strategy and use of devel jobs is decided tomorrow. But is ready for review now.

@mikaelarguedas mikaelarguedas added in review Waiting for review (Kanban column) and removed in progress Actively being worked on (Kanban column) labels Jul 9, 2018
@dirk-thomas
Copy link

👍 for all the referenced PRs.

@mikaelarguedas
Copy link
Author

addressed by connected PRs, closing

@mikaelarguedas mikaelarguedas removed the in review Waiting for review (Kanban column) label Jul 11, 2018
@mikaelarguedas
Copy link
Author

uncrustify 0.67 installed from brew on all MacOS nodes.
Set of ci Jobs testing both bouncy and current master: https://ci.ros2.org/job/test_ci_osx/
jobs 76 to 81

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants