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

Use 2.* in require-dev zendframework/zend-escaper in Zend\Debug\composer.json #6630

Closed

Conversation

samsonasik
Copy link
Contributor

No description provided.

@@ -17,7 +17,7 @@
"php": ">=5.3.23"
},
"require-dev": {
"zendframework/zend-escaper": "*"
"zendframework/zend-escaper": "self.version"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should actually require 2.*

@Ocramius Ocramius added this to the 2.3.4 milestone Nov 22, 2014
@samsonasik samsonasik changed the title Use self.version at Zend\Debug composer.json for require zend/escaper Use 2.* instead of self.version in all composer.json in all components Nov 22, 2014
@samsonasik
Copy link
Contributor Author

@Ocramius done, changed self.version to 2.* in all components, PR title updated too ;)

@Ocramius
Copy link
Member

Uh, no, that's not what I meant here :-(

It's just that Zend\Escaper is a security-sensitive component, therefore the user should always have the latest stable security fixes, without being pinned down to self.version.

@Ocramius Ocramius self-assigned this Nov 22, 2014
@samsonasik samsonasik changed the title Use 2.* instead of self.version in all composer.json in all components Use 2.* in require-dev zendframework/zend-escaper in Zend\Debug\composer.json Nov 22, 2014
@samsonasik
Copy link
Contributor Author

@Ocramius Ok, updated. I'm sorry for the missinterpretation. I've updated only for it and updated PR title too.

Ocramius added a commit that referenced this pull request Nov 29, 2014
@Ocramius Ocramius closed this in 5e74506 Nov 29, 2014
@Ocramius
Copy link
Member

@samsonasik no problem at all! Merged, thanks!

master: 5e74506
develop: 80170f6

gianarb pushed a commit to zendframework/zend-debug that referenced this pull request May 15, 2015
gianarb pushed a commit to zendframework/zend-debug that referenced this pull request May 15, 2015
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.

2 participants