Support multiple attachments

Drillster / drone-email

Drone plugin for sending email notifications

Apache License 2.0

44 stars 43 forks source link

Support multiple attachments #29

Closed michaelschlies closed 4 years ago

michaelschlies commented 4 years ago

fixes #28

mjwwit commented 4 years ago

Thanks for the PR! ❤️ I noticed you replaced the attachment property in favor of attachments. This, however, will break all existing usages of this plugin that use the former syntax. Would it be possible to support both?

michaelschlies commented 4 years ago

Updated

mjwwit commented 4 years ago

Lovely, it now builds again and sending multiple attachments works like a charm! I'll merge and it'll be released shortly after.