roslint

roslint copied to clipboard

Store lint output in the build tree

1

Creating an output file would be useful for users. Such a target should cause the lint command only to run again when its dependencies change.

jack-oquin

As pointed out in #12, some ROS code which follows PCL triggers a ton of these.

mikepurvis

Apart from [catkin_lint](https://github.com/fkie/catkin_lint), primarily check for: - two-space indentation. - lowercase function and macro names. - uppercase variable names.

mikepurvis

XML linter

1

Check for well-formedness, sane indentation.

mikepurvis

Scheme for silencing warnings/errors

1

Especially important for: - situations where a warning is acknowledged, but not actually valid. - maintainers who don't have the resources to clean everything up now, but would like to...

mikepurvis

avoid printing on stderr if linter successful

3

Hey there :wave: Thanks for this utility! I'm experimenting it to invoke `catkin_lint` as described in https://discourse.ros.org/t/ros1-now-is-a-great-time-to-add-catkin-lint-to-your-packages/36521 To be precise I'm invoking it like this `roslint_custom(catkin_lint -W2 --strict -q .)`)...

mikaelarguedas

Fix dict change during iteration

1

Without this PR, I got an exception from cpplint saying that the dict iterated over has been changed during iteration. The comment above my change says we operate on a...

peci1

Release new version for noetic

1

@mikepurvis can we get a new version with the setuptools change?

kjeremy