search

wetcat-studios / fortie

Laravel 5 package for Fortnox API
Apache License 2.0
19 stars 20 forks source link

Updated to Guzzle 7 #84

Closed stromgren closed 3 years ago

stromgren commented 3 years ago

Updated Guzzle dependency to version 7.

I've read the Guzzle documentation for upgrading and cannot find any potential issues: https://github.com/guzzle/guzzle/blob/master/UPGRADING.md

agoransson commented 3 years ago

Hey! Thanks for the PR! =)

I'm having a look how to make this pass, I think we need to also update the php version. Possibly we might need to update other dependencies to make this pass on php 7.

Edit: I'm thinking we should remove the php versions lower than 7.2 in .travis.yml. Is that something you can do in this PR? Thanks. =)

agoransson commented 3 years ago

Oh, and maybe add the lowest required php version to composer would be an excellent addition. Maybe something like this?

    "require": {
        "php": "^7.2"
    }
agoransson commented 3 years ago

Thanks!