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
added skipif for sqlite3 version > 3.24.0 in tests history #11401
Conversation
It doesn't work for me. |
Hi, I just did the alterations @toddrme2178 commented. I'm having problems with squashing all commits into one, but my changes are there. |
Do you mind if I do that for you and push on your branch ? Also side-note, you want to learn how to work on multiple branches, if you look at the top of this PR you can see that the request is to merge Usually you want to something along the line of:
And the the PR will ask to merge The workflow here is approximatif, you are free to modify it, but learning on how to manage multiple branches will be extremely helpfull Do not worry git is not obvious, but you'll learn your way around it. |
Hi @Carreau ! Thank you for the help with squashing the commits. I'm sorry for not answering you here earlier, I was trying to fix it for myself. If you can, please let me know how you did it, I'm not used just yet with the HEAD file. And thank you very much for merging my pull request! I'm still looking for simple issues to fix (I'm learning a lot from collaborating), if you find something please let me know. Thanks! |
No worries, I had an hour or two where I could work on that, so I did. Here is a gif that re-does the rebase/squash The I pushed with the --force-with-lease option on your branch. You're doing good so far. |
Backport PR #11401 on branch 5.x (Added skipif for sqlite3 version > 3.24.0)
This pull request is an attempt to solve issue 11372.
What I did:
I have no idea how to test this. I'de like some suggestions if possible!
Some observations: