Fix: code quality issues

[withshubh]

Description

Hi 👋
I ran the DeepSource static analyzer on the forked copy of this repo and found some interesting code quality issues. I have fixed a few of them in this PR.

Summary of changes

• Removed unused code
• Removed redundant return statement
• added .deepsource.toml

[FZambia]

@withshubh hello, thanks! But seems that I just fixed this in #417 while adding golangci-lint

[withshubh]

There are more issues in this repo which you might want to have a look. @FZambia

[FZambia]

@withshubh thanks, addressing file perm warning in #418 , other issues look minor enough. Thanks for pr - I'll take a closer look at DeepSource in the future.

[withshubh]

Thanks @FZambia 😊 If you want then I can create an issue for integration of DeepSource with the steps.

[FZambia]

At this moment I prefer to not integrate so such an issue will be pretty stale and probably closed. Integration is simple enough – if I decide this is a right move we'll do this.

BTW, does DeepSource.io support one-time repo analysis without full integration?

[withshubh]

At this moment I prefer to not integrate so such an issue will be pretty stale and probably closed. Integration is simple enough – if I decide this is a right move we'll do this.

BTW, does DeepSource.io support one-time repo analysis without full integration?

As of now there is no option to run one time analysis but we're working on CLI that will help you do so.

[withshubh]

Hey @FZambia 👋 I hope you're doing good.

Just wanted to share that DeepSource released CLI last month. I would really appreciate it if you could try it out and share your feedback with us. : )

[FZambia]

@withshubh hello, great to hear! I don't want to create account on deepsource to test this feature out.

[withshubh]

Okay @FZambia we won't be able to run analysis without an account :(