ui5-linter icon indicating copy to clipboard operation
ui5-linter copied to clipboard

Feature request: Ignore file

Open Leoooob opened this issue 1 year ago • 4 comments

Running the linter today, you can specify specific files, but not folders to lint. It doesn't make sense to expand file paths to allow folders, it does make sense that we should be able to set an ignore file to ignore generated files like the component.js for instance, or 3rd-party code and libraries.

The workaround right now is to specify every individual file in the project to be linted, which is a bit tedious but works for now.

I would prefer to be able to set a global ignore file for use across projects.

Leoooob avatar Jun 21 '24 13:06 Leoooob

Thanks for reporting. Excluding/Ignoring files is already planned and listed on our roadmap: https://github.com/SAP/ui5-linter/issues/21

RandomByte avatar Jun 24 '24 12:06 RandomByte

@Leoooob from your description I understand that it would be sufficient to only ignore on a file-level, not ignoring specific findings within a file, right?

matz3 avatar Jul 12 '24 13:07 matz3

Yes, that's correct.

Regards,

Leon

---- On Fri, 12 Jul 2024 14:08:09 +0100 @.*** wrote ----

@Leoooob from your description I understand that it would be sufficient to only ignore on a file-level, not ignoring specific findings within a file, right?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.

Leoooob avatar Jul 12 '24 13:07 Leoooob

Would be great to exclude directories as well, just not specific files like webapp/test/*

marianfoo avatar Aug 14 '24 15:08 marianfoo

This feature is implemented with https://github.com/SAP/ui5-linter/pull/264 and available with v0.4.0

flovogt avatar Sep 23 '24 10:09 flovogt