Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

--ignore does not behave as expected (reopening issue #222) #1349

Open
k1eran opened this issue Aug 17, 2023 · 1 comment
Open

--ignore does not behave as expected (reopening issue #222) #1349

k1eran opened this issue Aug 17, 2023 · 1 comment

Comments

@k1eran
Copy link

k1eran commented Aug 17, 2023

Describe the bug
Quoting stale issue #222

While there is an option called --ignore it does not seem to have the desired effect. No matter which paths I add files from those directories are still listed with errors.

To Reproduce
Steps to reproduce the behavior:

npx htmlhint gui/src/**/*.html --ignore="gui/src/assets/help/"

shows issues in gui/src/assets/help/en/some/file that should be hidden

Expected behavior
ignore supported as described in docs

Screenshots
N/A
Desktop (please complete the following information):
N/A

Smartphone (please complete the following information):
N/A

Additional context
See comment from groenroos commented on Apr 7, 2022.

This is still an issue, and should be re-opened. As it is a documented feature, it should also be prevented from going stale.

with 4 likes

@hyperupcall
Copy link

hyperupcall commented Oct 13, 2023

A related issue is that the treatment of single argument --ignore is naive with how it interprets the value. It interprets , as a glob separator, but , is used by globs in various expansions: --ignore='.{m,git,hg}ignore'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants