zopefoundation / zope.sqlalchemy

Integration of SQLAlchemy with transaction management
Other
32 stars 34 forks source link

Fix the tests by using an older transaction version. #19

Closed icemac closed 7 years ago

icemac commented 7 years ago

The last time the tests were green was running with transaction 1.4.4. So pin the to this version for now.

mmerickel commented 7 years ago

Pinning to an older transaction seems to be the worst possible solution to the problem. Obviously we need to fix the tests.

icemac commented 7 years ago

@mmerickel You are right. I wanted to bring the tests of the package into shape before someone can tackle #20 and #21 and show users that the package might be broken for newer transaction versions.