search

Smartling / api-sdk-php

SDK for integrating with the Smartling API. The Smartling API allows developers to upload language specific resource files and download the translations of those files for easy integration within their application. http://docs.smartling.com
Apache License 2.0
6 stars 15 forks source link

update for guzzle 7 #133

Closed vsolovei-smartling closed 2 years ago

vsolovei-smartling commented 2 years ago

guzzle 7 requires at least php 7.2, so I've updated the dependency to php 7.4. For tests to work, I had to update to phpunit 8. Some tests were broken, so I've introduced fixes.

vsolovei-smartling commented 2 years ago

All the system tests pass, right?

I would also like to check manually using for instance FAPI client. Specifically, I'm worrying about file upload calls. I briefly checked Guzzle 6 vs 7 compatibility changes and seems like no breaking changes in this part but let's check anyway.

All the tests pass. What exactly would you like to check?