search

ament / ament_lint

Apache License 2.0
38 stars 107 forks source link

[ament_pep257] ament_pep257 not showing errors as stderr #389

Closed mikramarc closed 2 years ago

mikramarc commented 2 years ago

Currently when running ament_pep257 command all the errors are shown as regular stdout output. This make's it hard to handle the errors, log correctly, visualize easily, etc. Seems like other loggers in ament do have that feature (i.e. actually outputting errors to stderr). Is there any desire/plan to change that anytime soon?

com2rng commented 2 years ago

@dirk-thomas (I suppose you are the maintainer of this repo, or at least the one with the largest amount of commits) since this issue happens in both master and foxy branches -- the latter I'm currently using--, how can we deal with this issue? I'm actually working with the boschresearch group and planning to contribute this fix back. Shall I create two different PR for fixes in master and foxy or is there a plan to update foxy to master soon?

[EDIT] I just found out that the maintainers of the pkg is @mjeronimo