Skip to content

Conversation

@Cito
Copy link
Member

@Cito Cito commented Jul 28, 2017

SQLAlchemy expects new-style classes.

So the manually mapped class must inherit from object, even though this is not necessary in Python 3.

SQLAlchemy expects new-style classes
@coveralls
Copy link

coveralls commented Jul 28, 2017

Coverage Status

Coverage increased (+0.4%) to 88.421% when pulling 1def397 on Cito:declarative_issue into c2da407 on graphql-python:master.

@coveralls
Copy link

coveralls commented Jul 28, 2017

Coverage Status

Coverage increased (+0.4%) to 88.421% when pulling 18bd114 on Cito:declarative_issue into c2da407 on graphql-python:master.

@coveralls
Copy link

coveralls commented Jul 28, 2017

Coverage Status

Coverage increased (+0.4%) to 88.421% when pulling 47941dc on Cito:declarative_issue into c2da407 on graphql-python:master.

@Cito Cito closed this Jul 28, 2017
@Cito Cito deleted the declarative_issue branch July 28, 2017 14:39
@Cito
Copy link
Member Author

Cito commented Jul 28, 2017

Sorry, the Travis config was not the problem. Closing this PR to create a better one.

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