fix(admin_manual): Explicitly specify which versions of MySQL #11801
Conversation
…gested Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
| | | - openSUSE Leap 15.5 | | ||
| | | - CentOS Stream | | ||
| +------------------+------------------------------------------------------------------------------------+ | ||
| | Database | - **MySQL** **8.0**/8.2/8.3 or MariaDB 10.3/10.5/**10.6**/10.11 (recommended) | |
| | | - openSUSE Leap 15.5 | | ||
| | | - CentOS Stream | | ||
| +------------------+------------------------------------------------------------------------------------+ | ||
| | Database | - **MySQL** **8.0**/8.2/8.3 or MariaDB 10.3/10.5/**10.6**/10.11 (recommended) | |
There was a problem hiding this comment.
I think we should only list the LTS version 8.0 and 8.4 (and add 8.4 to our testing matrix).
There was a problem hiding this comment.
The (recommended) was inline, to clarify that we recommend the 10.6 instead of any mysql/mariadb
But it seems that got lost already for mysql on the last bump. So let's hope the bolding is enough.
|
Hello there, We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process. Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6 Thank you for contributing to Nextcloud and we hope to hear from you soon! |
☑️ Resolves
Rather than say "8.0+" (which could lead to problems with deployments using untested versions of MySQL with Nextcloud) let's stick to the actual versions we test against.
Currently (in
master) that is, technically 8.0 and 8.3, but not the recently published 8.4. We are explicit with our other recommended databases so let's be consistent here too.Included 8.2 since, like for PostgreSQL, our automated test matrix skips it for efficiency (presumably) since it likely is covered by our 8.0 and 8.3 tests (it's out of scope to debate the pros/cons of that here).
🖼️ Screenshots
New:
Existing:
