DeepScan & Lighthouse Tuning and Fixes about binari HOT 6 OPEN

@OmarShezan I have updated the ESLint config file and fixed most of the warnings that could automatically be fixed. You should now be able to use ESLint to see most of the issues. DeepScan also runs its own rule set when merged which are un-related to ESLint which you will need to go to DeepScan to see, or install the DeepScan plugin in whatever Code Editor you are using.

As of now, all the errors/warning are related to ESLint and not DeepScan, so just calling npx eslint . should mostly do the trick.

from binari.

@vsan626 Go ahead and create a PR with the updated .eslintrc.js file and if the warnings go away, then great! Otherwise, we can work from there and see what needs to be fixed. I was aware of the React version warning, but decided to leave it to newcomers to fix (:

In the DeepScan score found here we can also see that there are two no-console warnings that you can also fix if you would like. After that I think that Binari's score should increase in the eyes of DeepScan; at least for now. There will definitely more warnings and errors to fix in the future as well.

Once you create a PR and will review and merge as soon as possible.

Thanks!

from binari.

is there a way to run DeepScan before creating a pull request? because at that point it's too late as ill have to delete my pull request and create a new one.

from binari.

After cloning this repo and running the code I see several warnings. When I run npx eslint . there was initially a warning saying React version was not specified, so I updated the .eslintrc.js file with the settings to fix that. After running npx eslint . again, no errors / warnings show up. Would the bugs / fixes that need to be handled come from the initial running of the code?

from binari.

I updated the About.js file by removing the console.log & changing the console.log inside the catch block to console.error. Noticed there may still be a warning with console.error after making a PR. Next idea would be to disable eslint's no-console within the About.js file.

from binari.

Hey , im a beginner and would love to contribute to this issue, can I be assigned for this?

from binari.