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
UI: Fixing warnings in react-app #7069
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Harsh Pratap Singh <harshpratapsingh8210@gmail.com>
Signed-off-by: Harsh Pratap Singh <harshpratapsingh8210@gmail.com>
Signed-off-by: Harsh Pratap Singh <harshpratapsingh8210@gmail.com>
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.
Maybe you could also remove CI=false
in the same PR to ensure that we don't get any warnings?
Signed-off-by: Harsh Pratap Singh <harshpratapsingh8210@gmail.com>
Signed-off-by: Harsh Pratap Singh <harshpratapsingh8210@gmail.com>
As I see |
Seems |
Yup! I am finding it difficult to fix those last warnings! I am quite stuck so either I close the PR or maybe get it merged as most of the warnings are fixed, maybe someone else might help fix those last warnings after the merge while the issue is still open. @yeya24 |
Changes
Fixes partially #7066
Verification
Some of the warnings are not fixed yet (WIP), others don't give warning when running
make react-app