python-lsp / python-lsp-server

Fork of the python-language-server project, maintained by the Spyder IDE team and the community
MIT License
1.96k stars 194 forks source link

Set return type to `None` for functions without returns #581

Closed agserrano3 closed 4 months ago

agserrano3 commented 4 months ago

πŸš€ Hi there!

I'm Toni GarcΓ­a, a colleague of @bagel897. Together, we're committed to improving code quality through automatically generated codemods. Our mission is to help maintain high standards by automating issue detection and resolution. πŸ”βœ¨

πŸŽ‰ We're excited to start collaborating with your team! In our upcoming PRs, you can expect to see:

These changes aim to be low-effort for the maintainers but highly effective in keeping the codebase clean and efficient.

πŸ”§ For this first PR, we've focused on enhancing code clarity by adding explicit return types for Python methods that do not return anything. This small change helps improve readability and maintainability of the code. πŸ’‘

πŸ“¬ FAQ

If you encounter any issues with the PR or want to discuss anything further, feel free to email me at toni@codegen.com. I'm always here to help and happy to talk to a human! πŸ™‹β€β™‚οΈ

Codegen β€’ What is Codegen?

Codegen unleashes the power of ⚑ GPT-4 code mods ⚑, fully integrated into your development workflow.

Learn more

Looking forward to your feedback and collaboration!

Cheers, Toni and Ellen

ccordoba12 commented 4 months ago

Hey @agserrano3, thanks for your contribution! Overall, it looks good to me so kudos to you and @bagel897 on the work you're doing at Codegen!

Please rebase or merge with develop to fix the merge conflicts. That way I'll be able to run our tests.

agserrano3 commented 4 months ago

thanks for you review @ccordoba12 , I have now solved the conflicts