-
Notifications
You must be signed in to change notification settings - Fork 448
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
chore(ci) fix prettier & eslint workflows #5740
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
No changes to documentation |
Component Testing Report Updated Feb 14, 2024 7:29 PM (UTC)
|
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.
Exciting!
package.json
Outdated
@@ -20,7 +20,7 @@ | |||
"check:test": "run-s test -- --silent", | |||
"check:types": "tsc --build", | |||
"chore:format:fix": "prettier --cache --write .", | |||
"chore:lint:fix": "pnpm check:lint -- --quiet --fix", | |||
"chore:lint:fix": "run-s check:lint -- --quiet --fix", |
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.
might be better to just use turbo directly here I think the args passing multiple times gets messed up
"chore:lint:fix": "run-s check:lint -- --quiet --fix", | |
"chore:lint:fix": "turbo run lint -- --quiet --fix", |
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.
Done. Thanks!
3b088d1
to
a539273
Compare
Description
Merging to next revealed a couple of issues with lint/prettier scripts. This should hopefully fix it.
Testing
Not sure if it's possible to properly test these before merging, unfortunately
Notes for release
N/A