Update css-to-inline-styles to v2.1

[Belphemur]

Update the test, remove the configuration that are not used anymore
Add config to set CSS files to be use directly in emails

[coveralls]

Coverage decreased (-5.4%) to 62.857% when pulling e763ee2 on Belphemur:master into e1869cb on fedeisas:master.

[ceesvanegmond]

@Belphemur Thank you for the addition! Why doesn't it pass on HHVM? Could you fix that?

[Belphemur]

It's the library itself, not the code here.

@tijsverkoyen has put HHVM as allow_failure on his test for now. You can see it here in the last build: https://travis-ci.org/tijsverkoyen/CssToInlineStyles/jobs/160315806

I can't do anything about this other than also asking travis to allow the failure.

[ceesvanegmond]

@Belphemur Could you add the

  allow_failures:
    - php: hhvm

to this merge request too?

[Belphemur]

@ceesvanegmond added 👍

[coveralls]

Coverage decreased (-5.4%) to 62.857% when pulling e84dff0 on Belphemur:master into e1869cb on fedeisas:master.

[ceesvanegmond]

@Belphemur Thx!

[ceesvanegmond]

@Belphemur Is this fully backwards compatible? Should we tag it on v1.6?

[Belphemur]

I would bump the major version since the behavior of the underlying lib also changed. It doesn't strip the style tag anymore.

Moreover the default config changed to add the possibility to set the css files and removed the other options.

Sent from BlueMail

On Sep 19, 2016, 16:42, at 16:42, Cees van Egmond notifications@github.com wrote:

@Belphemur Is this fully backwards compatible? Should we tag it on
v1.6?

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
#38 (comment)

[ceesvanegmond]

@Belphemur Right! Check it out!
https://github.com/fedeisas/laravel-mail-css-inliner/releases/tag/2.0

[Belphemur]

@ceesvanegmond Cheers!

Thanks for the quick update.