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
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants