Re: PR for UID EXPUNGE to achieve MOVE semantics
Thanks very much for your contribution and apologies for taking so long to get back to you.
I'd love to see support for MOVE.
That's a solvable issue. IMAPClient already patches out imaplib's Commands dict to add support for commands it doesn't already support. See https://github.com/mjs/imapclient/blob/1.x/imapclient/imapclient.py#L41-L65
It's ugly but it works. Future versions of IMAPClient probably won't depend on imaplib. When that happens the nasty patching can go.
I don't think many people use `use_uid=False`. What you've done looks sensible to me.
Thank you very much for taking the time to do the live test too.
Your approach looks good. Please submit a PR. Against 1.x is fine - I can take care of forward porting it if you like.
If you have the time to do a PR for MOVE as well, that would be amazing. It's worth having both MOVE and UID EXPUNGE support.
_command_and_check() should be giving you the response string so I'm not sure why you're not seeing anything. Perhaps set `debug=True` and see what the server is returning?
inbox@... :: http://menno.io/