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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added prettier #115

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

alexandercerutti
Copy link
Contributor

@alexandercerutti alexandercerutti commented Oct 18, 2022

Hey @rvagg, I know that in #108 we said to add a .editorconfig file, but sadly its options are limited for Javascript (for example, semi-colons). So I opted in for prettier to achieve better results.

I hope this will be still appreciated!

Let me know!

EDIT: Oh, I'm seeing right now that this seems to conflict with Standardjs checks 馃槼.
It seems that it would be required to use ts-standard to format in the same way also typescript declarations 馃 What should I do with this PR? Keep it, change to ts-standard o what else?

@rvagg
Copy link
Owner

rvagg commented Oct 19, 2022

I'm not familiar with ts-standard unfortunately, I welcome your input on that though. A shame about the .editorconfig options, but maybe standard --fix is good enough for folks with aggressive editor settings.

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

Successfully merging this pull request may close these issues.

None yet

2 participants