-
Notifications
You must be signed in to change notification settings - Fork 709
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
fix: fix code comments and variables errors #1711
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@intls is attempting to deploy a commit to the Coinbase Team on Vercel. A member of the Team first needs to authorize it. |
🟡 Heimdall Review Status
|
Hi Dev! @dschlabach Updated branch! Thanks! |
@intls - please take a look at the build - looks like you need to lint https://github.com/base/web/actions/runs/13703849650/job/38324204785?pr=1711 |
@dschlabach I don't know how to fix it, maybe better re-open PR? |
@dschlabach Unfortunately I can't solve this by myself, can you help me please? |
@intls are you able to build and run lint locally? |
@dschlabach Sorry, but I don't know how to do this. |
@dschlabach Tried run "yarn lint --fix" command but didnt help |
@dschlabach all done! Now lint ok! PR ready for review! Thanks! |
@intls - looks like CI is still failing. Would recommend you create a new PR because I'm not sure why this PR is causing that lint to fail ➤ YN0000: ./src/utils/hooks/useWithdrawalStatus.ts |
What changed? Why?
Notes to reviewers
How has it been tested?