Skip to content

test: update MySQL test matrix to use official images and add 9.0, 8.4 versions #7476

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

Merged
merged 2 commits into from
Jun 6, 2025

Conversation

enomotodev
Copy link
Contributor

  • Do only one thing
  • Non breaking API changes
  • Tested

What did this pull request do?

Summary

  • Update MySQL test matrix to use official Docker images
  • Add MySQL 9.0 and 8.4 to CI tests

Changes

  • Replace mysql/mysql-server:latest with mysql:8.0 for consistency with official images
  • Add MySQL 9.0 (latest) and 8.4 to ensure compatibility with newer versions
  • Keep MySQL 5.7 for backward compatibility testing

User Case Description

@@ -41,7 +41,7 @@ jobs:
mysql:
strategy:
matrix:
dbversion: ['mysql/mysql-server:latest', 'mysql:5.7']
dbversion: ['mysql:9.0', 'mysql:8.4', 'mysql:8.0', 'mysql:5.7']
Copy link
Member

Choose a reason for hiding this comment

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

Can you change it to

'mysql:9', 'mysql:8', 'mysql:5.7'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jinzhu
Updated as requested, thank you!

@jinzhu jinzhu merged commit 23c0d7c into go-gorm:master Jun 6, 2025
23 checks passed
@enomotodev enomotodev deleted the update-test-matrix branch June 6, 2025 04:18
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.

2 participants