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

added ext-iconv to require #128

Closed
wants to merge 1 commit into from

Conversation

samsonasik
Copy link
Contributor

make it require in the first place to avoid user get error when using it:

Call to undefined function Zend\Mail\Header\iconv_mime_decode()

@Ocramius
Copy link
Member

@samsonasik can you also bump the phpunit dep? I think that's what is causing the failure... Unsure about how much the mocking API is used though.

@samsonasik
Copy link
Contributor Author

@Ocramius I think it was because of sebastianbergmann/comparator@9d9ede5 that fixed at sebastianbergmann/comparator@2b7424b that released yesterday . I am closing and re-open to restart build.

@samsonasik samsonasik closed this Jan 30, 2017
@samsonasik samsonasik reopened this Jan 30, 2017
@samsonasik
Copy link
Contributor Author

@Ocramius travis green now ;)

@weierophinney weierophinney added this to the 2.8.0 milestone Feb 14, 2017
@weierophinney weierophinney self-assigned this Feb 14, 2017
@weierophinney
Copy link
Member

Assigning to 2.8.0 due to the new requirement.

weierophinney added a commit that referenced this pull request Feb 14, 2017
added ext-iconv to require
weierophinney added a commit that referenced this pull request Feb 14, 2017
weierophinney added a commit that referenced this pull request Feb 14, 2017
@weierophinney
Copy link
Member

Thanks, @samsonasik

@weierophinney
Copy link
Member

Merged to develop for release with 2.8.0

@Xerkus Xerkus mentioned this pull request Apr 11, 2017
@samsonasik samsonasik deleted the iconv branch March 6, 2018 07:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants