Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Remove duplicate: zend-stdlib is already required #5451

Closed
wants to merge 1 commit into from

3 participants

@ThaDafinser

zend-stdlib is required, so suggest is not needed

@Maks3w
Collaborator

The criteria for add a component in the require section is if the component has at least one explicit use statement with the required component.

Since there are 8 explicit use Zend\Stdlib\... then the composer.json is good as is.

@Maks3w Maks3w closed this
@ThaDafinser

Yeah, but its mentioned twice...
Required and suggest

@Maks3w
Collaborator

oh sorry

@Maks3w Maks3w reopened this
@Maks3w Maks3w was assigned
@Maks3w Maks3w referenced this pull request from a commit
Commit has since been removed from the repository and is no longer available.
@weierophinney weierophinney referenced this pull request from a commit
@weierophinney weierophinney Merge branch 'hotfix/5451' into develop
Forward port #5451

Conflicts:
	library/Zend/Filter/composer.json
7787623
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 2 deletions.
  1. +1 −2  library/Zend/Filter/composer.json
View
3  library/Zend/Filter/composer.json
@@ -23,8 +23,7 @@
"zendframework/zend-i18n": "Zend\\I18n component",
"zendframework/zend-uri": "Zend\\Uri component for UriNormalize filter",
"zendframework/zend-validator": "Zend\\Validator component",
- "zendframework/zend-crypt": "Zend\\Crypt component",
- "zendframework/zend-stdlib": "Zend\\Stdlib component"
+ "zendframework/zend-crypt": "Zend\\Crypt component"
},
"extra": {
"branch-alias": {
Something went wrong with that request. Please try again.