marrow / mailer

A light-weight, modular, message representation and mail delivery framework for Python.
MIT License
276 stars 62 forks source link

Next fixes #98

Closed homeworkprod closed 2 years ago

homeworkprod commented 3 years ago

Hey Alice, I finally managed to get a deeper go at the next branch (on Python 3.9, to which I just upgraded).

I came across a few errors right away for which I have provided what I think should be fixes (unless I misinterpreted your intention behind the code pieces in question).

Let me know what you think. :)

amcgregor commented 2 years ago

I'll have a gander at this shortly. Apologies for the insane delay.

homeworkprod commented 2 years ago

It seems to me you've applied pretty much these changes (including the LC → GE change) in 46b1c20a6672d51b9a9cf008aa4a581b1928903d already anyway.

amcgregor commented 2 years ago

Closing as already implemented. Temporary variable use instead of just switching which quotes were in use (d'oh!) but still effective.