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
documentation update for prettier-eslint #3
Comments
Glad you're finding this useful. I don't mind adding recipes to the docs. Could we simplify this first?
For a command configured to read from stdin, The script is not necessary unless you use it elsewhere. You could: {
"selector": "source.js",
"cmd": ["node", "$HOME/.yarn/bin/prettier-eslint", "--parser=babel", "--stdin"],
} This avoids spawning a subshell and runs |
Yes, I just confirmed this works as well. I was too eager to get it working, I forgot to use I made the following draft for you if you would like to use it. Sublime is a great text editor, and with
|
That's a big instruction. :) Where do you think such recipes should be located? It could make it harder to skim the readme for info about the plugin itself. Perhaps a folder with examples, listed from the readme... |
I started with a single paragraph, and next thing you know it was a full page :) But I think it covers pretty much any issues someone might have. I agree, this should be inside a |
My apologies for being slow, got a lot happening. 🙂 Sounds perfectly good. If you could PR this, that would be nice. 👍 |
Hello, thanks for the package. I was finally able to use prettier-eslint with sublime which before I had to run prettier through eslint but had to give up on some of the nice eslint rules (such as
space in parens
).Could you please mention this in your readme so others can possibly benefit from your package as well?
Here's the tweak I made:
a. create a new executable somewhere such as
/bin/beauty
with the following contentand set
/bin/beauty
as executable.chmod +x /bin/beauty
.b. the settings for fmt package
with the
cwd
mode set toauto
, it's reading the project.eslintrc
andprettier.config.js
as one would except.Thanks again,
The text was updated successfully, but these errors were encountered: