Topics/update to 2.2 #3

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

@till
till commented May 23, 2013

Update components and upped requirement to 5.4 — test suite fataled otherwise.

@weierophinney weierophinney commented on the diff May 23, 2013
composer.json
"require": {
- "php": ">=5.3.3",
- "zendframework/zend-escaper": "2.0.*",
- "zendframework/zend-eventmanager": "2.0.*",
- "zendframework/zend-http": "2.0.*",
- "zendframework/zend-i18n": "2.0.*",
- "zendframework/zend-log": "2.0.*",
- "zendframework/zend-mvc": "2.0.*",
- "zendframework/zend-session": "2.0.*",
+ "php": ">=5.4.0",
+ "zendframework/zend-escaper": "2.2.*",
@weierophinney
weierophinney May 23, 2013 collaborator

I'd argue "2.*", as it clearly works on the 2.0 series and above. I'll make that change on merge.

@till
till May 23, 2013

Yeah, good idea. Wasn't aware that 2.* works for composer! :) But yeah, since ZF uses semantic versioning this should do!

@weierophinney weierophinney commented on the diff May 23, 2013
.gitignore
@@ -3,3 +3,5 @@ vendor/
.vimproject
*.log
tests/log/
+/composer.phar
@weierophinney
weierophinney May 23, 2013 collaborator

leading slashes are not necessary.

@weierophinney weierophinney pushed a commit that referenced this pull request May 23, 2013
Matthew Weier O'Phinney [#3] Use 2.* as ZF component versions
- Also, remove "/" prefix from gitignore entries
8833035
@weierophinney weierophinney pushed a commit that closed this pull request May 23, 2013
Matthew Weier O'Phinney Merge branch 'hotfix/3'
Close #3
2cb0596
@till
till commented May 23, 2013

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment