Uh oh!
There was an error while loading. Please reload this page.
chore: Add EditorConfig for IDE whitespace - #1032
Conversation
janiceilene
commented
Nov 4, 2020
Thanks @nschonni! I'll get this triaged for review ✨ |
@nschonni just confirming, this |
nschonni
commented
Nov 5, 2020
Yes, the whitespace settings are picked up, so when you hit the Tab key, it inserts the indicated number of spaces, and will add the EOF marker if it is missing. {
"recommendations": ["EditorConfig.EditorConfig"]
}Didn't add that in this PR to keep the discussion focused, but I can add that in if you want. It looks like I had taken a quick look at what flipping to plain ESLint + Prettier would look like, but it reformatted so much I didn't want to submit it out of the blue |
JasonEtco
left a comment
There was a problem hiding this comment.
This looks great to me, thanks for the addition @nschonni! If I was going to write a .editorconfig file, this is exactly what I'd write 📝 I'll leave it to @chiedo or @janiceilene to actually merge this once they're ready, but 👍 from an eng perspective. ✨
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. |

Why:
Allows IDEs and GitHub web editor to detect the whitespace settings when editing files https://editorconfig.org/
What's being changed:
Add a basic .editorconfig file, but don't run any normalizaton
Check off the following: