-
-
Notifications
You must be signed in to change notification settings - Fork 7
feat: add biome vscode settings & use biome instead of eslint #17
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
Conversation
✅ Deploy Preview for tsky ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
I tried to use the |
package.json
Outdated
| }, | ||
| "nano-staged": { | ||
| "*.{js,ts,cjs,mjs}": ["biome check --write --"] | ||
| "*.{js,ts,cjs,mjs,json}": ["biome check --write --"] |
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.
Would it support for markdown as well?
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.
I think they're still working on that.
.github/workflows/ci.yml
Outdated
|
|
||
| - name: 📝 Check format | ||
| run: pnpm format:check | ||
| run: pnpm format |
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.
Does the normal format throw errors if something is not formatted?
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.
Yep, this is the equivalent to eslint without --fix
bd3b7dd to
9e42fc5
Compare
No description provided.