Skip to content

refactor: change methods not using its bound instance to staticmethods#8

Open
deepsource-autofix[bot] wants to merge 1 commit intomainfrom
deepsource-autofix-85ba8eab
Open

refactor: change methods not using its bound instance to staticmethods#8
deepsource-autofix[bot] wants to merge 1 commit intomainfrom
deepsource-autofix-85ba8eab

Conversation

@deepsource-autofix
Copy link
Copy Markdown
Contributor

The method doesn't use its bound instance. Decorate this method with @staticmethod decorator, so that Python does not have to instantiate a bound method for every instance of this class thereby saving memory and computation. Read more about staticmethods here.

The method doesn't use its bound instance. Decorate this method with `@staticmethod` decorator, so that Python does not have to instantiate a bound method for every instance of this class thereby saving memory and computation. Read more about staticmethods [here](https://docs.python.org/3/library/functions.html#staticmethod).
@deepsource-io
Copy link
Copy Markdown
Contributor

deepsource-io bot commented Dec 22, 2024

Here's the code health analysis summary for commits 34c3480..c3be5ca. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Python LogoPython❌ Failure
❗ 1 occurence introduced
🎯 2 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

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.

0 participants