Should SuppressionFilter use OS-independent paths

[Pavel Baranchikov]

Hello.

We have some code, that we do not want to check by Checkstyle, that's why we use

SuppressionFilter.

I found a nasty behaviour, that files attribute specifies paths in OS-specific manner.

Take a look at http://stackoverflow.com/questions/13874582/what-is-the-correct-syntax-to-suppress-specific-checkstyle-rules-on-certain-file
I think, it's awful to use constructions, like this:

    files="my[\\/]super[\\/]package[\\/]name"

I suggest to go by Ant's path. Expect all the files to have unix-like path separators (/) and apply them correctly on all paths, including Windows.

Any objections here?

Thanks in advance,
Pavel.

[Michal Kordas]

Hi,

No objections from me. Ant paths are pretty much standard in tools similar to Checkstyle. Could you please create issue on GitHub?

The only concern is backward compatibility. Would it require 7.0 release?

Thanks,
Michal

--
You received this message because you are subscribed to the Google Groups "checkstyle-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email to checkstyle-dev...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[Pavel Baranchikov]

Submitted https://github.com/checkstyle/checkstyle/issues/2270

Thanks.