-
Notifications
You must be signed in to change notification settings - Fork 150
Codestyle unification #187
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
|
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.
Thanks! Everything is style/cosmetic except the image opt stream right? I'll test this in a bit then we can merge.
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 there's a couple of uncommitted files
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@sladg is there anything special I need to do to get the autoformat to work? I have eslint and prettier plugin installed in my VSCode, but when I save a file, it doesn't add the commas. |
@khuezy disable prettier plugin. Only eslint is needed as prettier does not have configuration file in workspace so it's fighting with eslint most likely. |
I think I have some weird VSCode settings in my global VSCode settings somewhere in my /Library/Application Support/Code/User. It might be better to create a |
My VSCode broke... so I can't really test this PR smh... |
I was using typescript 5.2.2 and the eslint lib didn't support it yet. The fix got deployed ~30m ago and everything is working after I reinstall to pick up new packages. See: typescript-eslint/typescript-eslint#7155 (comment) We should upgrade to latest TS too |
Do you mind adding
That will autoformat to the style on save. |
I also had to make some changes to server-adapter.ts. Let me know if you want me to push the changes up or not. |
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.
LGTM Thanks!
* feat(eslint): configuration added for eslint and general styling
What this does?