search

o4oren / Ad-Hoc-Email-Server

An ad-hoc disposable temporary mail server. Create and use ad hoc disposable mailboxes for testing or any other purpose.
Apache License 2.0
385 stars 91 forks source link

drive-by language review #8

Closed jetmore closed 5 years ago

jetmore commented 5 years ago

Nice project. Just driving by and saw some typos and consistency issues. There are probably some more to make but I just stuck to the inarguable issues. I didn't touch it because it's not clear to me what you would prefer, but I really recommend picking a consistent form of referring to your project. On this page I see 'AHEM server', 'AHEM', and 'Ahem mail server'.

o4oren commented 5 years ago

Thanks. I approved all your changes! Oren

On Sun, Oct 27, 2019 at 2:17 PM John Jetmore notifications@github.com wrote:

Nice project. Just driving by and saw some typos and consistency issues. There are probably some more to make but I just stuck to the inarguable issues. I didn't touch it because it's not clear to me what you would prefer, but I really recommend picking a consistent form of referring to your project. On this page I see 'AHEM server', 'AHEM', and 'Ahem mail server'.

You can view, comment on, or merge this pull request online at:

https://github.com/o4oren/Ad-Hoc-Email-Server/pull/8 Commit Summary

  • drive-by language review

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/o4oren/Ad-Hoc-Email-Server/pull/8?email_source=notifications&email_token=AFTF65RA6UPMVUKPEIVQRN3QQWBEPA5CNFSM4JFRN642YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HUTD67A, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFTF65R3MARSEACLOV6ZGV3QQWBEPANCNFSM4JFRN64Q .