-
-
Notifications
You must be signed in to change notification settings - Fork 412
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
Validation Warning: Unknown option ( ... ) was found in the config root. #482
Comments
We're seeing the same issue. May need to revert to an earlier release. |
Same issue, the bug seems to have appeared in 3d0ccb2. I got rid of the warning by pinning lint-staged to version 7.1.2 ( |
I had to nuke the Edit: I was able to narrow it down to |
This is the PR which introduced the issue - jestjs/jest#6802, working on the fix. |
(Jest maintainer here) Is it a bug in jest-validate? |
I'd say it should have been a breaking change.. |
If it correctly reports that invalid config is invalid, I don't think it should be a breaking change. |
At least in our case, we relied on checking only the top level keys. The way we are using |
Hmm, now I'm seeing
when using latest version from master :-/ @sudo-suhas is this something we're aware of? |
@okonet Could you please share the output of |
@sudo-suhas interesting. Here is the output:
What I don't get is why lint-staged using the wrong version?
although we're requiring the right one? https://github.com/okonet/lint-staged/blob/4832b4c079e8d148f23d79abe09148f04fd0b3ea/package.json#L39 |
My guess is that there are a lot of edge cases when using package managers which are hard to reproduce and therefore hard to fix. Nuking the |
Also, after rebasing did you run |
@sudo-suhas I was hoping that this is why we have
Nuking |
😔 I know what you mean. |
What's the solution for this? I'm literally using the same config from the readme and getting the warning. {
"lint-staged": {
"linters": {
"*.{js,scss}": ["some command", "git add"]
},
"ignore": ["**/dist/*.min.js"]
}
}
|
Try removing and reinstalling node_modules |
What fixed it was upgrading to 7.3.0, thanks. |
Description
I have tried every way I can think of to set up the config in package.json and keep getting this warning.
I have tried:
and
all three options throw the same validation warning on pre-commit. What am I doing wrong here?
Debug Logs
expand to view
Environment
lint-staged
: 7.2.0The text was updated successfully, but these errors were encountered: