Closed justinnoel closed 1 year ago
@justinnoel Sorry I didn't see this sooner, there really seems to something wrong with my GitHub notification settings as I didn't get a notification for this. I would be very happy to accept a PR for this! Would it make sense to have both configs?
@jonasmerlin Ugh.... I took forever to get back to this.
It's really best to only choose .editorconfig
OR Prettier.
@justinnoel Alright, then let's go with Prettier I'd say!
@justinnoel Runnign npm run format
formats files according to the astro rules by now. So I'll close this issue. If you're missing something, feel free to reopen.
Would you be willing to accept a PR for adding either a Prettier configuration or editorconfig for this project?
Either would make it much easier for others to contribute without constantly making changes to the spaces, tabs, etc.
I'd be happy to submit a PR using either based on your preferences. I personally prefer prettier, but it does require more tooling on the developer side. Most IDEs are already 100% aware and compliant with editorconfig.
May I suggest these configurations based on your existing code:
Prettier
editorconfig
NOTE: In each, I have use tabs as the indentation vs spaces. Tabs have some significant accessibility benefits over spaces. Also, I used double quotes as your code already seems to have them.
P.S. editorconfig does not officially recognize
quote_type
but many IDEs respect this setting.