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
#9120 improves mail services by making it python3 compatible #11898
base: trunk
Are you sure you want to change the base?
Conversation
I found the open file in text mode so this conversion can go back to how I found it.
some py3 tests enabled and working, some py3 tests not upgraded to py3
for more information, see https://pre-commit.ci
@Zectbumo Thanks for this contribution. A bit embarrassing that it was necessary, but I hope we can get CI green on this so that we can land and get to the point where the CLI at least works out of the box again 😬 |
@Zectbumo any chance you could have a look at getting these tests to pass so we can land this fix? |
I am getting a "The logs for this run have expired and are no longer available." message so I can't tell if this is something simple or expected to fail. I can do some simple fixes to pass checks but my patch is expected to not pass all tests. The patch I submitted is a better-than-nothing patch since getting to true python 3 compatibility probably requires some overhaul of a component or a big decision to be made. |
If you can just resolve the conflicts and push something, we can see where we're at; and we can selectively un-skip whatever tests pass now which failed previously, we don't need everything to pass as long as the stuff we expect to pass, passes |
Sure, I just need access to the logs. I am getting a "The logs for this run have expired and are no longer available." from github. |
Yes, the test run here is too old; the logs have been deleted, github doesn't keep them around forever. Once you've resolved the conflicts, you can merge in trunk, and when you push the branch with trunk merged in, you you'll get a fresh CI run where you can see the logs. |
ah, I see. I'm resolving conflicts. I understand now. I'll look into this. |
Thanks a bunch! |
Fixes #9120
this makes smtp, pop3, using maildir work and makes the twistd mail tap command work at least with the defaults or basic configuration.
some python 3 tests were enabled and some passed, others still failed. it is a WIP and I am commiting as is since it is useable.
more python3 upgrades are still needed to make fully functional and more py3 tests need to be fixed and enabled.