Shorten line in docstring #450
Conversation
|
@sea-bass can you remind me what the And is the error here something ado with the ros ctesting pipeline? Complaining because it didn't find a test report (because no tests were run?)? |
That's right, Rolling and Jazzy. These 2 versions have in common that they use Ubuntu 24.04, so maybe something related to that? |
|
This looks like it's Noble or ROS2 machinery that's flipping out on the fact that there are no tests in the ROS PR jobs now (which served mainly to smoke test deb builds). I didn't want to setup any test machinery in the ROS PR jobs - that's just duplicate configuration that's a pain to maintain for a package in which I used a non-ROS environment for developing. I did some digging, but not getting any quick answers. Left a question at colcon/colcon-core#678 too. One thing we could try is to explicitly add pytest to that setup.py along with a fake test that always passes to point it at. |
|
Looks like their suggestion was exactly your idea -- seems like that's the plan! |
|
Merging this for the formatting fix |
My recent change seems to have broken the linter due to line length. Quick fix!