open-rmf / rmf_task

RMF library for managing task allocations
Apache License 2.0
22 stars 20 forks source link

Fix uncrustify tests for humble #68

Open orensbruli opened 2 years ago

orensbruli commented 2 years ago

Bug report

Looks like a minor uncrustify discrepancy: https://github.com/open-rmf/rmf_task/runs/8285620339?check_suite_focus=true#step:5:1738

Required information:

Description of the bug

https://github.com/open-rmf/rmf_task/runs/8285620339?check_suite_focus=true#step:5:1738

  2: --- src/rmf_task/TaskPlanner.cpp
  2: +++ src/rmf_task/TaskPlanner.cpp.uncrustify
  2: @@ -541 +541,2 @@
  2: -        return {};
  2: +        return {}
  2: +      ;

Steps to reproduce the bug

Reproducible in CI runs. No tested in local.

Expected behavior

Tests pass.

Actual behavior

Test fails

orensbruli commented 2 years ago

I think I found an issue related to this: https://github.com/uncrustify/uncrustify/issues/3191