go-admin icon indicating copy to clipboard operation
go-admin copied to clipboard

[Proposal] Potential code quality issues found

Open ghost opened this issue 5 years ago • 3 comments

I forked this repo a while ago and ran a DeepSource analysis on it. DeepSource found a variety of different issues categorized based on their types and severity which you can view here.

Brief description - Anti-Patterns: 125 Bug Risks: 45 Performance issues: 20 Security Issues: 234

You can find a detailed description and fixes for some of them here.

Please let me know what issues you'd like to focus on fixing and I'd be happy to take a look into it. Also, you can choose to hide certain types of issues too (if you wish to ignore them or you believe it is a false positive). I'd also be happy to send a patch with the DeepSource configuration file with the required fixes.

You can find the required configuration file for DeepSource here.

ghost avatar Aug 13 '20 10:08 ghost

@chsatyap Thanks for the feedback.

chenhg5 avatar Aug 15 '20 02:08 chenhg5

这个是什么检测工具,这个先进啊

gerrywp avatar Aug 20 '20 03:08 gerrywp

@gerrywp It's a static analyzer tool. I'd be happy to point you to further resources if required!

ghost avatar Aug 24 '20 14:08 ghost