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

run tests against MySQL 5.6 #232

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SingingBush
Copy link

Not sure if you'd want to run mysql-native against MySQL 5.6 as well but thought I'd add it just in case. Feel free to decline this one. I don't have a need for it.

@schveiguy
Copy link
Collaborator

Needs a copy-paste from the new integration targets.

@schveiguy
Copy link
Collaborator

How do we specify that it's OK if it fails? I don't mind testing against 5.6, but I don't want it to prevent progress.

@SingingBush
Copy link
Author

There is continue-on-error. To be fair though I'm not bothered about checking mysql 5.6, only pushed this in case it was considered helpful to someone. Am happy to also close the PR, the builds are already fairly robust

@schveiguy
Copy link
Collaborator

What if we split this into its own workflow yaml file? That way at least it will not show that integration tests failed if the 5.6 runs failed? Then we can at least easily turn it off if needed.

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