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

Bump db_version to 8.0 #61

Merged
merged 2 commits into from
Jul 22, 2024

Conversation

ironprogrammer
Copy link
Contributor

What is this?

WordPress's minimum MySQL version was recently bumped to 5.5.5, which means this plugin no longer works with WordPress trunk. Let's bump the reported DB version to 8.0 to make it last a while.

Version bump was done here:
WordPress/wordpress-develop@e1265a3

Also see this for a similar change:
WordPress/sqlite-database-integration@768f8cc

Why do this?

Even though there is a canonical plugin furthering this work, this dropin remains an easy, scriptable way to use SQLite on a new WordPress install without a MySQL server requirement.

@calebsmithdev
Copy link

With 6.5 out in production now - any information on getting this merged?

@MikeD78yw
Copy link

It would be nice to have it merged, it's a pain to manually change the content of db.php every time.

@matapatos
Copy link

Same here.

@dingo-d
Copy link
Contributor

dingo-d commented Jun 18, 2024

Can confirm that this solves the issue when running wp-pest on WP 6.5+ Can we merge it? I have a fork where I've added .gitattributes file and updated the test matrix a bit, so I can also make a PR with the improvements, but only if it will be merged.

@aaemnnosttv any words on if you'll be able to merge this?

@aaemnnosttv
Copy link
Owner

Thanks for the PR @ironprogrammer and apologies for the delay on this. I just (unknowingly) released https://github.com/aaemnnosttv/wp-sqlite-db/releases/tag/v1.3.2 which bumps this to 5.5.5 but I'm happy to update it for consistency with the core integration 👍

@aaemnnosttv aaemnnosttv merged commit 5d8af6b into aaemnnosttv:master Jul 22, 2024
6 checks passed
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.

6 participants