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. #369

Draft
wants to merge 9 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

  • Ignore Paths in PR and Push Events
    Some paths in our project have been marked to be disregarded in pull request and push events, decreasing irrelevant warnings in our automatic process.

  • Packages and PHP Version Updates
    We updated the required software packages and the minimum PHP version in our composer.json file, ensuring the latest features & security enhancements.

  • Test File Method Signature Updates
    Various testing files had updates in their setUp() and tearDown() methods. These methods set up and conclude our tests, now their signatures (how they're invoked) are renewed to ensure better testing alignment and practices. This includes tests for active data provider, fixtures, active records, relations, batch queries, caching, command, database, session, and migrations.

  • Removal of compatibility.php
    The bootstrap.php file no longer includes compatibility.php and it has been deleted, cleaning up unnecessary elements in our repository.

  • Update to Mongo Tests
    The methods setUp() and tearDown() in the MongoDB test files have now included return types. This specifies what kind of result the methods should return, making the code more understandable and less error-prone. Also, the test assertions using assertContains were replaced with assertStringContainsString for better string occurrence 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