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.

Sign up for GitHub

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

Jump to bottom

Adds the appropriate return type for the method or function #1162

Closed
wants to merge 2 commits into from
Closed

Adds the appropriate return type for the method or function #1162

wants to merge 2 commits into from

Conversation

deepsource-autofix[bot]
Copy link
Contributor

@deepsource-autofix deepsource-autofix bot commented Sep 29, 2021 •
edited by sayak-sarkar
Loading

Fixes #1161

@sayak-sarkar sayak-sarkar requested review from riginoommen and ghanlohar and removed request for riginoommen September 29, 2021 18:19
@sayak-sarkar sayak-sarkar added chore Internal things, technical debt, and to-do tasks to be performed. enhancement New feature or request labels Sep 29, 2021
@deshmukhmayur
Copy link
Member

Closing due to inconsistent styling and to avoid accidental merge. I think deepsource can create another PR when the analyzer rules are changed.

@deshmukhmayur deshmukhmayur deleted the deepsource-fix-2915e9f3 branch June 16, 2022 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@riginoommen riginoommen riginoommen approved these changes

@ghanlohar ghanlohar Awaiting requested review from ghanlohar

Assignees

@riginoommen riginoommen

@ghanlohar ghanlohar

Labels
chore Internal things, technical debt, and to-do tasks to be performed. enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

(JS-0306) Missing explicit return and argument types
4 participants
@deshmukhmayur @riginoommen @sayak-sarkar @ghanlohar