Skip to content

Pin Uncrustify version to 0.67#2

Closed
jacobperron wants to merge 1 commit intomasterfrom
strict_uncrustify_version
Closed

Pin Uncrustify version to 0.67#2
jacobperron wants to merge 1 commit intomasterfrom
strict_uncrustify_version

Conversation

@jacobperron
Copy link
Copy Markdown
Contributor

Do not use newer versions.

With this change we can be sure the same version of Uncrustify is being used on all platforms.

Resolves ros2/rosidl#340

Do not use newer versions.
@jacobperron jacobperron added the in progress Actively being worked on (Kanban column) label Dec 18, 2018
@jacobperron
Copy link
Copy Markdown
Contributor Author

jacobperron commented Dec 18, 2018

Test on macOS with uncrustify v0.68 installed:
Build Status (cppcheck errors expected)

@jacobperron
Copy link
Copy Markdown
Contributor Author

Closing in favour of #3

@jacobperron jacobperron deleted the strict_uncrustify_version branch December 19, 2018 22:54
@jacobperron jacobperron removed the in progress Actively being worked on (Kanban column) label Dec 19, 2018
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.

1 participant