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

Moved ext-intl to required instead of suggested. #126

Merged
merged 2 commits into from
Nov 18, 2019

Conversation

adamculp
Copy link
Contributor

@adamculp adamculp commented Nov 18, 2019

Because the extension is not included in all default builds of PHP, and is required for translate usage.

Provide a narrative description of what you are trying to accomplish:

  • Are you fixing a bug?
    • Because the php-intl extension is required for even the most simply use of translate, and not all installs of PHP include it by default. Having it in the suggested area, when most usage requires it, seemed inappropriate.

Because the extension is not included in all default builds of PHP, and is required for translate usage.
composer.json Outdated Show resolved Hide resolved
weierophinney added a commit that referenced this pull request Nov 18, 2019
Moved ext-intl to required instead of suggested.
weierophinney added a commit that referenced this pull request Nov 18, 2019
@weierophinney
Copy link
Member

Thanks, @adamculp; merged to develop for release with 2.10.0.

@michalbundyra michalbundyra added this to the 2.10.0 milestone Nov 18, 2019
@weierophinney weierophinney merged commit f3902c9 into zendframework:master Nov 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants