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

Rename links-mysql8 driver to links-mysql? #1182

Closed
jamescheney opened this issue Jul 3, 2023 · 2 comments · Fixed by #1189
Closed

Rename links-mysql8 driver to links-mysql? #1182

jamescheney opened this issue Jul 3, 2023 · 2 comments · Fixed by #1189

Comments

@jamescheney
Copy link
Contributor

I thought we decided at the meeting last week to, in effect, discard the current links-mysql driver and rename links-mysql8 to links-mysql (while still using ocaml-mysql8 under the hood), but just-merged #1178 just removes links-mysql without renaming the links-mysql8 driver.

I think we should do this rename, thus ensuring that the links-mysql package continues to be the right thing to use to connect to MySQL independent of what links version one is using.

@jamescheney
Copy link
Contributor Author

@frank-emrich ahs there been any further discussion of this? I think if we do this it should be done before the next release so that we don't wind up with the extra OPAM package.

@frank-emrich
Copy link
Contributor

frank-emrich commented Oct 10, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants