Skip to content

BUGFIX - Remove with_metaclass method#9

Merged
msiemens merged 1 commit intomsiemens:masterfrom
dinkopehar:master
May 22, 2020
Merged

BUGFIX - Remove with_metaclass method#9
msiemens merged 1 commit intomsiemens:masterfrom
dinkopehar:master

Conversation

@dinkopehar
Copy link
Copy Markdown
Contributor

This pull references this issue.

If I understood your code correctly, with_metaclass is method for compatibility between python 2 and python 3 for creating meta classes.

Since it's removed from TinyDB (v4.0.0), I refactored code, and run unit tests successfully. If I made a mistake or did something wrong, please notify me. Thank you.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 95.946% when pulling eed4dc2 on dinko-pehar:master into 526524e on msiemens:master.

@msiemens msiemens merged commit deecfb8 into msiemens:master May 22, 2020
@msiemens
Copy link
Copy Markdown
Owner

Thanks for your pull request! I've merged it and will release a new version of tinydb_serialization in the next week or so.

@dinkopehar
Copy link
Copy Markdown
Contributor Author

Great ! I'm glad this got resolved well

@dagraham
Copy link
Copy Markdown

Thanks very much!

@msiemens
Copy link
Copy Markdown
Owner

tinydb-serialization v2.0.0 has been published which supports TinyDB v4 :)

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.

4 participants