Skip to content

change methods not using its bound instance to staticmethods#36

Merged
mrayanasim09 merged 1 commit into
mainfrom
deepsource-autofix-08939ded
Aug 7, 2023
Merged

change methods not using its bound instance to staticmethods#36
mrayanasim09 merged 1 commit into
mainfrom
deepsource-autofix-08939ded

Conversation

@deepsource-autofix
Copy link
Copy Markdown
Contributor

@deepsource-autofix deepsource-autofix Bot commented Aug 7, 2023

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.


This change is Reviewable

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).
@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
python-projects ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 7, 2023 9:03pm

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 7, 2023

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

@mrayanasim09 mrayanasim09 merged commit bef1ce1 into main Aug 7, 2023
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Aug 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant