-
Notifications
You must be signed in to change notification settings - Fork 850
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
Update ts, eslint, prettier #963
base: main
Are you sure you want to change the base?
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 @rin-st!!
Just added a couple of comments
Also in tailwind.config.js
I see an error but on main
it doesn't seem to appear:
Also maybe its good chance to solve this warnings in .eslintrc
files ?
present at both nextjs and hardhat.
Also pushed a minor commit not related to this PR fixing the types of useScaffoldEventHistory
Thanks for the great comments!
It turns out that by default Plus:
|
Description
@typescript-eslint/eslint-plugin
from nextjspackage.json
since it's already included toeslint-config-next
"plugins": ["@trivago/prettier-plugin-sort-imports"]
to avoid warnings. From docscheck-types
script for hardhatNote:
eslint
v9 is still not supported by nextjs, so I used last v8 versionTo test take code snippet here
Additional Information
Related Issues