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

zend-hydrator v1.1 compatibility #149

Merged
merged 1 commit into from
Jul 27, 2016

Conversation

thomasvargiu
Copy link
Contributor

@thomasvargiu thomasvargiu commented Jul 13, 2016

zend-hydrator v1.1 seems to be compatible with zf-hal, so I downgraded the minimum version to allow ^1.1.
I don't know why it was updated to ^2.2.1.
Anyone see some problem?

@michalbundyra
Copy link
Member

@thomasvargiu yes, it's good to me 👍

Now tests run over stdlib 2.7 not only v3!

This requirement caused also other issues - for example this one:
zfcampus/zf-content-negotiation#74
because on ZF2 application (with stdlib v2) - zf-content-negotiation 1.2.1 was installed but without zf-hal 1.4.0

@weierophinney
Copy link
Member

Just spot checked the travis results, and these two lines show the change:

Looks good! Thanks, @thomasvargiu !

@weierophinney weierophinney added this to the 1.4.1 milestone Jul 27, 2016
@weierophinney weierophinney self-assigned this Jul 27, 2016
@weierophinney
Copy link
Member

I don't know why it was updated to ^2.2.1

Oversight; lots of moving parts. Thanks for catching this!

@weierophinney weierophinney merged commit 8e50cbc into zfcampus:master Jul 27, 2016
weierophinney added a commit that referenced this pull request Jul 27, 2016
weierophinney added a commit that referenced this pull request Jul 27, 2016
weierophinney added a commit that referenced this pull request Jul 27, 2016
weierophinney added a commit that referenced this pull request Jul 27, 2016
Forward port #149

Conflicts:
	CHANGELOG.md
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants