-
Notifications
You must be signed in to change notification settings - Fork 623
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
Add PHP 7.1 and 7.2 to Travis build matrix #565
Conversation
Thanks @emirb, This looks good at first glance. The failing tests are not related to your PR. I have added this to our backlog for final review and merge. With Best Regards, Elmer |
Hello @emirb, Just wondering why you closed this PR? It's still on our backlog for review and merge. With Best Regards, Elmer |
Reopened now — I just didn't want it open for 2 months without any activity so I cleaned up my profile a little bit last week. |
Thanks @emirb! |
Thanks for taking the time to code review @ssiddhantsharma! |
Total of 4 months now |
Thank you for your continued patience @emirb, We are continuing to work through the over 1,000 Hacktoberfest PRs and we are currenlty down to less than 300 so far. I don't see any issue with merging this one when it comes up next on our backlog. With Best Regards, Elmer |
Thanks @thinkingserious! :) |
No description provided.