-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch internal parser to email-addresses #20
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.
'
I see that you've got some upstream PRs merged. Is this just waiting on an upstream release now? |
No. The last issue I posted on the repo is still an issue.
Regardless, it doesn't allow us to do SMTPUTF8 in Haraka, but I'll post on
that issue why.
…On Mon, May 22, 2017 at 2:26 PM, Matt Simerson ***@***.***> wrote:
I see
<https://github.com/jackbowman/email-addresses/pulls?q=is%3Apr+is%3Aclosed>
that you've got some upstream PRs merged. Is this just waiting on an
upstream release now?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#20 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAobY2LX-JOGrVmutWV5Q8HpoAt6VQc_ks5r8dNSgaJpZM4MKUCh>
.
|
But those are not yet merged/published? And so neither can this, yet? |
Never mind. I fixed the lint issues but other issues remain. |
Correct. I'm still waiting on those changes being merged. I ran the tests against a local version.
When they merge them I'll post on this PR.
… On Jun 16, 2017, at 10:28 PM, Matt Simerson ***@***.***> wrote:
Never mind. I fixed the lint issues but other issues remain.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
…rfc2822 into use_email_addresses
Codecov Report
@@ Coverage Diff @@
## master #20 +/- ##
==========================================
- Coverage 90.05% 87.82% -2.24%
==========================================
Files 1 1
Lines 171 115 -56
Branches 44 29 -15
==========================================
- Hits 154 101 -53
+ Misses 17 14 -3
Continue to review full report at Codecov.
|
$ npm publish
|
This branch switches the internal parser to using email-addresses.
It is currently waiting on a few changes required to the upstream library, and may require a few more tweaks once those go in, as such, tests currently fail so hold off on merging.