chore: use biome for linting/formatting #38
Conversation
| }, | ||
| "javascript": { | ||
| "formatter": { | ||
| "quoteStyle": "single" |
There was a problem hiding this comment.
FWIW, if you want to match our primary styleguide, you'd want a formatter config more like:
{
"formatter": {
"indentStyle": "space",
"indentWidth": 2,
"lineWidth": 120
},
"javascript": {
"formatter": {
"quoteStyle": "single",
"bracketSameLine": true,
"bracketSpacing": false
}
}
}Up to you if you want them to match or don't really mind either way (i'm all for just using defaults nowadays)
There was a problem hiding this comment.
Thanks! I'm not heavily opposed to matching our style... but to your point, I just really want defaults where it makes sense. Except double quotes, I'll just never understand that as the default.
There was a problem hiding this comment.
Whitespace looks a little wild lol.
My only hesitation is that this seems like a departure from our existing tools stack. Is this a direction the tools / leverage team is on board with? I'm more conservative with this change because it's the sample app, meaning future apps may copy/pasta this config.
eslint 9 jumped the shark and I'm tired of managing 148 dependencies