J7mbo / twitter-api-php

The simplest PHP Wrapper for Twitter API v1.1 calls
MIT License
1.82k stars 802 forks source link

travis: add more recent PHP versions #260

Closed TomasVotruba closed 6 years ago

J7mbo commented 6 years ago

Thanks for the PR! The platform constraints could probably be simplified to >= 5.3 rather than a list we'd have to update all the time.

If you change that, 1 commit, into dev branch, I'll be happy to merge.

TomasVotruba commented 6 years ago

This is travis.yml, not composer.json. I think it works different

Also this is first of few commits I plan to add

J7mbo commented 6 years ago

Ah, I missed this travis part and thought it was composer. Thanks! Also, please put this onto the dev branch! :)

TomasVotruba commented 6 years ago

Sure. Could you set dev it as default to prevent this in the future?

How do they actually differ? Looks like they have same activity.

TomasVotruba commented 6 years ago

Not relevant anymore. I'm closing to clean my opened PRs.

Feel free to cherry pick anything you need.