Skip to content

Mention Slim Framework factories #159

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

Merged
merged 1 commit into from
Oct 17, 2016
Merged

Mention Slim Framework factories #159

merged 1 commit into from
Oct 17, 2016

Conversation

tuupola
Copy link
Contributor

@tuupola tuupola commented Oct 14, 2016

This is accompanying documentation PR for php-http/message#53 if it gets approved. Running make html builds docs successfully.

@dbu
Copy link
Contributor

dbu commented Oct 15, 2016

👍

php-http/message#53 requires slim 3.5 at least. should we mention somewhere in the doc that the code is not compatible with older versions of slim? or is there no reason for the factories to go with slim 3.5 and it could be ^3.0? then the lowest tests would ensure it works with slim 3.0 and the other tests would still look at the newest released 3.x version of slim.

@tuupola
Copy link
Contributor Author

tuupola commented Oct 15, 2016

There should be no problems with 3.0 but I will test it first.

@dbu dbu merged commit 4cc854f into php-http:master Oct 17, 2016
@dbu
Copy link
Contributor

dbu commented Oct 17, 2016

and that concludes the feature. thanks a lot for your contribution @tuupola !

@tuupola
Copy link
Contributor Author

tuupola commented Oct 17, 2016

Glad I can help. This project is awesome! Just finished writing an SDK to an internal API we are using. Can not believe how easy it was.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants