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

Fix code quality issues #83

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix code quality issues #83

wants to merge 1 commit into from

Conversation

withshubh
Copy link

@withshubh withshubh commented Dec 15, 2020

Hi 👋

I ran DeepSource analysis on my fork of this repository and found some interesting code quality issues which you might want to fix. Take a quick look at all the issues caught by DeepSource for this repository here
You can also have a look at the configuration file I used for DeepSource Analysis.

DeepSource is a code review automation tool that detects code quality issues and helps you to automatically fix some of them. You can use DeepSource to track test coverage, Detect problems in Dockerfiles, etc. in addition to detecting issues in code.
All the features of the DeepSource are mentioned here.

I'd suggest you integrate DeepSource since it is free for Open Source projects forever.

Using DeepSource to continuously analyze your repository

  • Merge this PR. I have included a .deepsource.toml in this PR, which you can use to configure your analysis settings.
  • Install DeepSource on your repository here.
  • Activate analysis here.

@withshubh
Copy link
Author

Hey @bradtraversy 👋

Please have a look 👀

@Steven-zion
Copy link

Am getting an error of cannot read property findOne of undefined. Can this be outdated

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

Successfully merging this pull request may close these issues.

3 participants