colcon / colcon-core

Command line tool to build sets of software packages
http://colcon.readthedocs.io
Apache License 2.0
99 stars 44 forks source link

Suppress flake8 A005 in existing colcon API #636

Closed cottsay closed 3 months ago

cottsay commented 3 months ago

Rather than suppress A005 completely, we can ignore it in our existing API to prevent new A005 violations from appearing.

I couldn't find a good way to suppress only A005 in the files themselves.

clalancette commented 3 months ago

The tests are failing here because we need https://github.com/colcon/colcon-core/pull/635 , but since that has a circular dependency with this, I've approved this one anyway.