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

added ext-iconv to require#128

Closed
samsonasik wants to merge 1 commit intozendframework:masterfrom
samsonasik:iconv
Closed

added ext-iconv to require#128
samsonasik wants to merge 1 commit intozendframework:masterfrom
samsonasik:iconv

Conversation

@samsonasik
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

Thanks, @samsonasik

@weierophinney
Copy link
Copy Markdown
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