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

Option to suppress Cannot process specified file: it’s ignored error #348

Open
mfisher87 opened this issue Apr 22, 2024 · 2 comments
Open

Comments

@mfisher87
Copy link

mfisher87 commented Apr 22, 2024

Subject of the feature

I would like to be able to ignore or suppress the Cannot process specified file: it’s ignored error because I want to run AlexJS with pre-commit (https://pre-commit.com).

I'm using this hook: https://github.com/mfisher87/alexjs-pre-commit-mirror

Problem

The way pre-commit automates running programs is by passing changed files to that program. If, for example, I ignore *.csv in .alexignore, then run pre-commit after changing foo.csv, pre-commit will call alex foo.csv, and AlexJS will complain and cause the check to fail even though everything is fine:

data/acadia.csv
  1:1  error  Cannot process specified file: it’s ignored

Expected behaviour

I'd like to pass an arg, for example --suppress-ignored-file-error, to convert this error to a warning, or to suppress it entirely.

Alternatives

@wooorm
Copy link
Member

wooorm commented Apr 23, 2024

There’s a silentlyIgnore option for the engine, which is used in the CLI.
A PR adding a --silently-ignore flag that turns that on is welcome!

@mfisher87
Copy link
Author

Thanks for that useful context! I'll try and make some time, but I do have a long backlog right now. If anyone else is looking for this feature, don't wait on me. I'll post here when I am able to get started.

iamnewton added a commit to theholocron/configs that referenced this issue Oct 16, 2024
Based on get-alex/alex#348, any pre-commit
hooks are going to throw an error for files ignored.

Signed-off-by: Newton <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants