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
Add options for different html
/ hbs
quote styles in quotes
rule
#2754
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for starting on this!
html
/ hbs
quote styles in quotes
rule
What's the status of this? |
I noticed that this hadn't been picked up in a while, so I just made some changes to resolve the comments and push it across the finish line. I think it's good to go now - lmk if you have any additional comments. @bmish |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
I think I did this right?
Fixes #617
Personally I think the default config should be as described in the issue above with double only used for HTML attributes. Also the config forces the need to specify both
hbs
andhtml
but it could also just default todouble
for whichever one of those is missing. So for example you could just do{ hbs: 'single' }
to get the same functionality, essentially still usingdouble
as a default.Some code/messages etc in here could probably use better words. Maybe moustache/curlies instead of handlebars for example. And HTML Attributes instead of just HTML. I'm too tired to deal with naming things.
I think it might need more tests too?