Skip to content

Comments

fix: ignore test report directories from linting#1096

Merged
susnux merged 1 commit intomainfrom
fix/noid/exclude-test-output-dirs
Jul 23, 2025
Merged

fix: ignore test report directories from linting#1096
susnux merged 1 commit intomainfrom
fix/noid/exclude-test-output-dirs

Conversation

@Antreesy
Copy link
Contributor

Noticed while linting.
If there's more known libraries to add (from what we're using), let me know

@Antreesy Antreesy requested review from ShGKme and susnux July 23, 2025 09:00
@Antreesy Antreesy self-assigned this Jul 23, 2025
@Antreesy Antreesy added bug Something isn't working 3. to review labels Jul 23, 2025
@susnux
Copy link
Contributor

susnux commented Jul 23, 2025

Additional idea: Check if there is a .gitignore and read that - the format is the same as per docs so we could handle much more cases :)

@susnux susnux merged commit 2150173 into main Jul 23, 2025
7 checks passed
@susnux susnux deleted the fix/noid/exclude-test-output-dirs branch July 23, 2025 12:13
@ShGKme
Copy link
Contributor

ShGKme commented Jul 23, 2025

Additional idea: Check if there is a .gitignore and read that - the format is the same as per docs so we could handle much more cases :)

It always diapointed me that ESLint doesn't ignore everything from gitignore by default...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants