Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

remove confusing print() #138

Merged
merged 1 commit into from
Sep 26, 2022
Merged

remove confusing print() #138

merged 1 commit into from
Sep 26, 2022

Conversation

tsasao
Copy link
Contributor

@tsasao tsasao commented Aug 5, 2022

No description provided.

@so-rose
Copy link

so-rose commented Aug 21, 2022

+1 for this, I was running a local script to send mails out recently, and it complained about lacking httpx and aioredis - even though I needed neither (or did I?)

A neat enhancement that may prevent some source code perusing!

@sabuhish
Copy link
Owner

Thanks for the MR, I forgot to remove it while testing!

@sabuhish sabuhish merged commit 7e724a3 into sabuhish:master Sep 26, 2022
@Feijo
Copy link

Feijo commented Sep 30, 2022

Hi @sabuhish when is this going to be released?

@sabuhish
Copy link
Owner

sabuhish commented Oct 2, 2022

Hi @Feijo, I will release it today

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants