-
Notifications
You must be signed in to change notification settings - Fork 282
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's good to see typo fixes in this Pull Request, but this fix on line 423 may be incorrect? Perhaps the original line should be split into two lines for clarity, i.e. from:
"Number of reconnections: %d.\nNumber of emails quarantined: %d.\n" \
... to ...
"Number of reconnections: %d.\n" \ "Number of emails quarantined: %d.\n" \
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I thought I had fixed this one. I'll have a look.
I had thought of splitting the line but wasn't certain you'd agree. If you do agree, i'll do that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, and split the lines - both for clarity and avoiding codespell false positives.
37fe15f to
14c90c9
Compare
No description provided.