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

Make it work with sqlalchemy>=1.4.0 #51

Merged

Conversation

triptec
Copy link
Contributor

@triptec triptec commented Nov 13, 2020

This is what I have had to do to make it work with sqlalchemy 1.4.0b1

* add test workflow

* Fix flaky test: Sleep one sec after first commit so that the second commit does not happen the same second

* Use SkipTest to get rid of <TestResult has no addExpectedFailure method, reporting as passes>
…here's a better way to get _entity_zero, I haven't spent any time looking
@flying-sheep
Copy link
Contributor

@michaelbukachi so this is a priority, SQLAlchemy 1.4 is already released.

@michaelbukachi
Copy link
Collaborator

@michaelbukachi so this is a priority, SQLAlchemy 1.4 is already released.
Alright. Reviewing.

This was referenced Mar 18, 2021
@michaelbukachi michaelbukachi merged commit 9c5b5c2 into absent1706:master Mar 18, 2021
@michaelbukachi
Copy link
Collaborator

Hey @flying-sheep, do you mind testing off the master branch? I'll create a release if no issues are found.

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.

None yet

3 participants