Skip to content

Conversation

iammix
Copy link

@iammix iammix commented Nov 21, 2024

No description provided.

@ebrahimraeyat
Copy link
Owner

Hello. Thank you so much.

please add test for it. I don't know how can I set automatic test ci on it. Do you know?

@iammix
Copy link
Author

iammix commented Nov 25, 2024

Hello!

I’ll go ahead and add a test for it. As for setting up automatic test CI, I’d recommend using a service like GitHub Actions. It’s free for most use cases and integrates directly with your repository.
If you’d like, I can help you set up the CI or provide a starter configuration. Let me know!

@ebrahimraeyat
Copy link
Owner

sure. I will be happy if you can help me. thanks.

you changed a lot of style in my code. is it necessary? what formating software you use?

@iammix
Copy link
Author

iammix commented Nov 26, 2024

Sure, I’m happy to help! 😊

Regarding the changes, the formatting was done automatically by my IDE, which uses autopep8 to comply with the PEP 8 guidelines. Some adjustments might look different, but they are mainly for consistency and readability based on Python's style guide.

If you’d prefer, I can easily revert the changes or modify them to better match your preferences. Just let me know what works best for you!

@ebrahimraeyat
Copy link
Owner

Thanks. I prefer to separate each argument of function in one line. If you can, please back the format. thanks.

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.

2 participants