ament / ament/ament_lint

--exclude negates the target path

Aperta
#499 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
more-information-needed
Lingua principale
Python
Stelle
50
Fork
129
Merge medio
3g 10h
PR unite (30g)
3

Descrizione

```shell
# Correct: lints all files in src
ament_cpplint src

# Bug: lints all files in current directory (possibly still excluding some things, I didn't verify that)
ament_cpplint --exclude "some/path.cpp" src

# Correct: workaround that does lint all files in src, excluding the path
ament_cpplint --exclude "some/path.cpp" -- src
```
I assume the way argparse is used to define 'exclude' is defaulting the 'paths' to '.'
I wouldn't be surprised if similar problems exist for other linters.

Only tested this on Iron.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.