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

[4.4] Pin version MySql Version in Tests to 8.0 #43420

Merged
merged 3 commits into from
May 5, 2024

Conversation

rdeutz
Copy link
Contributor

@rdeutz rdeutz commented May 2, 2024

Summary of Changes

Pinned the mysql version to 8.0

Testing Instructions

Check drone if integration tests are working

Actual result BEFORE applying this Pull Request

integration tests failing

Expected result AFTER applying this Pull Request

integration tests are working

.drone.yml Outdated Show resolved Hide resolved
@wilsonge
Copy link
Contributor

wilsonge commented May 2, 2024

Given the bugfix/LTS version is 8.0 and 8.3 is the 'innovation track' without any LTS surely we just want to be recommending people use the 8.0 series for now and test against that? https://dev.mysql.com/doc/mysql-installation-excerpt/8.3/en/which-version.html

At minimum we should test 8.0 and latest 8.x because most hosts will stick on 8.0 for that reason

@richard67
Copy link
Member

At minimum we should test 8.0 and latest 8.x because most hosts will stick on 8.0 for that reason

@wilsonge Did you mean latest 8.0.x? And what do you suggest for this PR here? Change it to 8.0 for now?

@wilsonge wilsonge enabled auto-merge (squash) May 5, 2024 11:58
@richard67 richard67 changed the title [4.4] Pin version MySql Version in Tests to 8.3 [4.4] Pin version MySql Version in Tests to 8.0 May 5, 2024
@wilsonge wilsonge merged commit 14746b7 into joomla:4.4-dev May 5, 2024
3 checks passed
@rdeutz rdeutz deleted the Pin-MySql-Version-to-8.3 branch July 30, 2024 07:07
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.

None yet

5 participants