prettier doesn't use eslintConfig when modules are installed globally

i have installed prettier and eslint extensions in vscode, and enable prettier.eslintIntegration setting. and i have installed eslint and eslint-config-airbnb-base modules globally in NPM. but prettier used prettierOptions rather than eslintConfig when i format codes. example codes: after formating: however when i installed prettier-eslint-cli module globally in NPM. then run prettier-eslint .\src\index.js. it worked as expected. see below. i think there are some issues when prettier resolving eslintConfig if modules are installed globally. hope it be fixed.

[ndstephens]

Unless I missed it in the docs, I didn’t see anywhere that in order for prettier.eslintIntegration:true to work you need to have all your eslint dependencies installed locally, not globally. It makes sense now, and I understand why they should be installed locally per project instead of globally. However when I was trying to get this to work for the first time, and happened to have everything installed globally, I’ve been struggling to make it work for a long time…until I came across this issue post.

Maybe in your docs where you write:

prettier.eslintIntegration (default: false) - JavaScript and TypeScript only Use prettier-eslint instead of prettier. Other settings will only be fallbacks in case they could not be inferred from ESLint rules.

you could add something like:

prettier-eslint already installed with extension. All other eslint dependencies need to be installed locally

Again, maybe i just missed that info, or the assumption that a dev will know to install all eslint dependencies locally is so strong that you feel it doesn’t need to be stated (i’m still a bit of a newb).

And to be clear…I say none of this mockingly or sarcastically. I love the extension and all the hard work that has gone into this. Just offering up some unsolicited advice from a newish dev.

[ndstephens]

just created pull request #629

open to all adjustments, improvements, and general verification of the text. I believe it’s accurate, but not 100% certain