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

Add rdbms matrix for wider compatibility testing #196

Open
wants to merge 1 commit into
base: 0.7.x
Choose a base branch
from

Conversation

WyriHaximus
Copy link
Member

@WyriHaximus WyriHaximus commented Mar 21, 2024

This change set introduces a rdbms matrix with the purpose of increasing insurance this package works with up to the latest MySQL/MariaDB versions.

Note that due to MySQL's versioning there are no v6 and v7.

Refs: #194

@WyriHaximus WyriHaximus added this to the v0.7.0 milestone Mar 21, 2024
@WyriHaximus WyriHaximus force-pushed the 0.7.x-add-rdbms-matrix-for-wider-compatibility-testing branch 5 times, most recently from 3d2d16b to b9f6703 Compare March 21, 2024 21:01
Copy link
Contributor

@SimonFrings SimonFrings left a comment

Choose a reason for hiding this comment

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

Interesting approach, but I'm not sure if we should add this to the same MySQL job here. You suggested to test this with mariadb:10 across all supported PHP versions, but I don't think this is necessary here. Looking further, if we want to support additional databases in different versions, this would result in a huge (and completely confusing) test matrix in GitHub actions.

I think we should test mysql across all supported PHP versions, but for mariadb we could also test against the latest PHP version(s) in a different job to avoid overloading this.

What do you think?

@WyriHaximus
Copy link
Member Author

Interesting approach, but I'm not sure if we should add this to the same MySQL job here. You suggested to test this with mariadb:10 across all supported PHP versions, but I don't think this is necessary here. Looking further, if we want to support additional databases in different versions, this would result in a huge (and completely confusing) test matrix in GitHub actions.

We could introduce MariaDB in a follow up PR.

I think we should test mysql across all supported PHP versions, but for mariadb we could also test against the latest PHP version(s) in a different job to avoid overloading this.

Givin that MariaDB is a fork by the creator on MySQL, we could flip a could and prefer one of them?

What do you think?

To be brutally honest: Test all permutations; but that is a personal preference that I enforce in my own packages and projects, so I'll make the adjustment for this PR.

This change set introduces a `rdbms` matrix with the purpose of increasing insurance this package works with up to the latest MySQL/MariaDB versions.

Note that due to MySQL's versioning there are no `v6` and `v7`.

Refs: friends-of-reactphp#194
@WyriHaximus WyriHaximus force-pushed the 0.7.x-add-rdbms-matrix-for-wider-compatibility-testing branch from b9f6703 to 249c5fe Compare April 23, 2024 13:09
@WyriHaximus
Copy link
Member Author

@SimonFrings Pushed the changes

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