Add base Biome config #203
Conversation
| "arrowParentheses": "always", | ||
| "bracketSpacing": true, | ||
| "indentWidth": 2, | ||
| "indentStyle": "space", |
There was a problem hiding this comment.
🟡 This will be hell to migrate. We're using tab in all of our FE projects. Should we stick to default tab configuration?
There was a problem hiding this comment.
I'd prefer to keep space-based indentation for packages and BE, though. can we do tab indentation as a FE-specific override?
There was a problem hiding this comment.
Sure we can. Is BE using spaces by default (at Lokalise)?
There was a problem hiding this comment.
yup, with the exception of Autopilot
There was a problem hiding this comment.
Should we also create an override for AP(backend)? or do we want to migrate to space?
There was a problem hiding this comment.
last time I ran a poll on this topic, 100% of AP engineers who participated selected the "I don't care" option, so it doesn't look like we need to maintain two separate sets of rules, we can just reformat it once and apply it within open PRs as well
There was a problem hiding this comment.
Sounds great to me! 🙏 thanks for sharing Igor 🙇
Changes
This provides a strawman proposal to kickstart the definition of our biome rules, any suggestions to adjust the rules would be most welcome.
Checklist
major,minor,patchorskip-release