Skip to content
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

Potential code quality issues found #759

Closed
pnijhara opened this issue Jun 23, 2020 · 0 comments
Closed

Potential code quality issues found #759

pnijhara opened this issue Jun 23, 2020 · 0 comments

Comments

@pnijhara
Copy link

pnijhara commented Jun 23, 2020

I have forked this repo and ran DeepSource analysis on it. The analysis reported over 2600+ issues where there were:

  • Antipatterns: 1900+ (out of which 254 can be auto fix by DeepSource)
  • Bug-risks: 410 (out of which 33 can be auto fix by DeepSource)
  • Performance issues: 142 (out of which 108 can be auto fix by DeepSource)
  • Security issues: 120 (out of which 63 can be auto fix by DeepSource)

A more detailed report of the Analysis can be found here => https://deepsource.io/gh/pnijhara/NeMo/

I have also added a .deepsource.toml file for the analysis to run in my forked repo which you can check here

Do let me know which of these issues you would consider high priority, and which ones would you want to ignore or consider false-positive? I can send a PR with some fixes if you want.

You can also activate this analysis on your own, and have it triggered on each PR if you integrate DeepSource in the repo.

Disclaimer: I work for DeepSource

dcurran90 pushed a commit to dcurran90/NeMo that referenced this issue Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants