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.

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

[Codegen] 🔧 Convert Function Calls to Use Named Kwargs #582

Conversation

agserrano3
Copy link
Contributor

@agserrano3 agserrano3 commented Jul 18, 2024

🚀 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:

📈 Improvements in code quality.
🤖 Automation of common issues.
🛠️ Resolutions for existing problems.
These changes aim to be low-effort for the maintainers but highly effective in keeping the codebase clean and efficient.

🔧 For this second PR, we have focused on adding the name of the parameter into the function calls. That will make the code easier to read and more error-proof 👌

📬 FAQ

If you encounter any issues with the PR or want to discuss anything further, feel free to email me at [email protected]. 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

@agserrano3 agserrano3 changed the title 🔧 Convert Function Calls to Use Named Kwargs [Codegen] 🔧 Convert Function Calls to Use Named Kwargs Jul 18, 2024
@agserrano3
Copy link
Contributor Author

@ccordoba12 I have now solved the conflicts with this PR (plus fixed some ruff issues). It should be ready to go :)

@agserrano3 agserrano3 closed this by deleting the head repository Oct 3, 2024
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.

1 participant