Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added possibility to use imagine new metadata api #413

Merged
merged 2 commits into from
May 22, 2014

Conversation

digitalkaoz
Copy link
Contributor

removed the hardcoded version pinning on imagine 0.5

using the new metadata-api in autorotatefilter.

should be bc.

@@ -17,7 +17,7 @@
},
"require": {
"php": ">=5.3.2",
"imagine/Imagine": "0.5.*",
"imagine/Imagine": "~0.5",
Copy link
Collaborator

Choose a reason for hiding this comment

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

@digitalkaoz are you sure the feature available from 0.5.0 version. I'd say we have to add ~0.5.x restriction, something like that

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the feature is only available in 0.6 but i wanted to keep bc (and dont force people to upgrade imagine (0.6. is still dev))

Copy link
Collaborator

Choose a reason for hiding this comment

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

sorry, I have written a comment and just after that saw that bc layer. Looks good to me.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

@digitalkaoz ~0.5,<0.7.0

@makasim
Copy link
Collaborator

makasim commented May 22, 2014

@digitalkaoz could you rebase?

@makasim makasim added this to the v1.0.0 milestone May 22, 2014
@digitalkaoz
Copy link
Contributor Author

done

@makasim
Copy link
Collaborator

makasim commented May 22, 2014

@digitalkaoz still is not mergeable. are you sure you rebase the latest master?

@digitalkaoz
Copy link
Contributor Author

ok, now it should...lets wait for travis

@@ -98,9 +99,13 @@
<argument type="service" id="liip_imagine.cache.signer" />
</service>

<service id="liip_imagine.meta_data.reader" class="%liip_imagine.meta_data.reader.class%" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

make it private.

makasim added a commit that referenced this pull request May 22, 2014
added possibility to use imagine new metadata api
@makasim makasim merged commit 92e6495 into liip:master May 22, 2014
@makasim
Copy link
Collaborator

makasim commented May 22, 2014

@digitalkaoz thanks

@digitalkaoz digitalkaoz deleted the metadata_reader branch May 22, 2014 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Level: New Feature 🆕 This item involves the introduction of new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants