Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

fix(dependencies): upgrading nodemailer #1755

Closed

Conversation

lirantal
Copy link
Member

@lirantal lirantal commented Apr 9, 2017

Looking at the new docs for NodeMailer it seems we can upgrade safely as the same simple SMTP transport creation remains compatible.

@lirantal lirantal self-assigned this Apr 9, 2017
@simison
Copy link
Member

simison commented Jun 13, 2017

@lirantal you can now rebase + re-run Travis for this as Node.js v4 isn't being tested anymore (#1768).

Nodemailer v3+ requires Node.js v6+ (changelog)

@lirantal
Copy link
Member Author

yep, I'll definitely do that, thanks for helping triaging these!

@codydaig
Copy link
Member

@simison Didn't see your rebase comment when I first read it and was messing around with it.

@lirantal I was just curious (and bored) and created a separate PR for this so feel free to merge that one or rebase this and merge this one. I'm indifferent either way.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants