Remove Safe

[rodnaph]

Summary

Removes thecodingmachine/safe from the dependencies as suggested for PHP 7.1 support.

Test plan (required)

The existing test suite should remain green.

Checklist

• Have you followed the guidelines in our CONTRIBUTING guide?
• Have you lint your code locally prior to submission (make lint)?
• Have you written new tests for your core changes, as applicable?
• Have you successfully ran tests with your changes locally (make tests)?
• I have squashed any insignificant commits
• This change has comments for package types, values, functions, and non-obvious lines of code

[rodnaph]

@gulien What's the deal with the codecov/travis-ci build steps, should they be running? Looks like the travis one passed on the last PR merged, but something has happened with a migration to travis-ci.com (I'm not familiar with this tool sorry).

https://travis-ci.com/github/thecodingmachine/gotenberg-php-client

[gulien]

That's because Travis uses a "credit" system, even for OSS, and thecodingmachine organization does not have credits anymore.

I guess I'll need to migrate to GitHub actions 😄

[rodnaph]

I guess I'll need to migrate to GitHub actions 😄

I'll raise another PR... 🤣

[rodnaph]

@gulien I've moved the build to Github Actions, but it's not running on this PR as I believe you need to specifically enable it to run on forks in the repo settings. You can see the builds run on my repo though:

https://github.com/rodnaph/gotenberg-php-client/actions/runs/449204547

There are a bunch of failed builds in there too so you can see the build fails when steps fail. A tweak was needed to resolve alternative DNS for the Gotenberg service in the ClientTest.

[rodnaph]

@gulien Your suggested Github Actions changes are on this branch now.

eg. https://github.com/rodnaph/gotenberg-php-client/runs/1621077191?check_suite_focus=true

[gulien]

👍

[gulien]

Thank your very much @rodnaph 😉