tesseract-robotics / tesseract

Motion Planning Environment
http://tesseract-docs.rtfd.io
Other
275 stars 88 forks source link

Find collision components optionally for environment test suite #941

Closed marip8 closed 1 year ago

marip8 commented 1 year ago

Changes per commit message

Levi-Armstrong commented 1 year ago

I really do not like this for the unit tests, because it is to easy for this to get broken you will never know that the tests are not being ran as part of the pipeline. Why is this needed?

marip8 commented 1 year ago

Why is this needed?

I almost never build FCL locally, so when I also build unit tests locally this causes my whole build to fail

Levi-Armstrong commented 1 year ago

I understand but I would rather this not be default behavior for unit tests.