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

Raise min version to PHP 7.3, check compatibility PHP 8.3. #84

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

terabytesoftw
Copy link
Member

Q A
Is bugfix?
New feature? ✔️
Breaks BC?

@what-the-diff
Copy link

what-the-diff bot commented Sep 27, 2023

PR Summary

  • Changes in Workflow File
    The workflow file was updated to trigger upon any new changes in the 'master' branch. Moreover, it now utilizes the latest PHPUnit testing setup and additional extensions for image manipulation have been added for more comprehensive testing scenarios.

  • Composer Specification Modification
    The minimum required PHP version has been increased to 7.3 for better performance and compatibility. Also, a more recent version of PHPUnit is required, allowing the use of the latest testing features.

  • Updates in Test Files
    The testing functions were updated in several files to assure compatibility with the recent changes. The'void' return type was added in setup and teardown methods of various testing files, enhancing readability and aligning with coding best practices. One of the tests now anticipates a distinct error message for invalid driver inputs.

  • File Deletion
    In an effort to keep our codebase clean and up-to-date, an obsolete compatibility file has been deleted.

These changes aim to modernize our infrastructure and improve the efficiency and reliability of our testing.

@codecov
Copy link

codecov bot commented Sep 29, 2023

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant