Skip to content
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

Define lowest deps #290

Merged
merged 2 commits into from
May 26, 2020
Merged

Define lowest deps #290

merged 2 commits into from
May 26, 2020

Conversation

j0k3r
Copy link
Collaborator

@j0k3r j0k3r commented May 18, 2020

Define the lowest deps possible for the lib to work:

Also:

  • add a new build with --prefer-lowest for composer, so the lib is tested with the lowest deps possible
  • rise minimum version of PHP to 5.6

Define the lowest deps possible for the lib to work:
- tecnickcom/tcpdf has a security advisory before 6.2.22 (see https://packagist.org/packages/tecnickcom/tcpdf/advisories?version=2463879)
- atoum/atoum doesn't work with version 3.0.0 (got few issues) and the version 2.8.0 doesn't work with PHP 7

Also:
- add a new build with `--prefer-lowest` for composer, so the lib is tested with the lowest deps possible
- rise minimum version of PHP to 5.6
.travis.yml Outdated Show resolved Hide resolved
.travis.yml Outdated Show resolved Hide resolved
@j0k3r j0k3r requested a review from k00ni May 26, 2020 07:48
Copy link
Collaborator

@k00ni k00ni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, can be merged.

@j0k3r j0k3r merged commit f7fac8e into smalot:master May 26, 2020
@j0k3r j0k3r deleted the fix/define-lowest-deps branch May 26, 2020 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants