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.

Sign up for GitHub

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

Jump to bottom

Enum8 and Enum16 supported #263

Merged

xzkostyan merged 3 commits into xzkostyan:master from akurdyukov:master

Sep 28, 2023

Contributor

akurdyukov commented Sep 26, 2023

fixes Enum + Clickhouse 1.4 = crash #192

Checklist:

Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
Add or update relevant docs, in the docs folder and in code.
Ensure PR doesn't contain untouched code reformatting: spaces, etc.
Run flake8 and fix issues.
Run pytest no tests failed. See https://clickhouse-sqlalchemy.readthedocs.io/en/latest/development.html.

akurdyukov added 3 commits

September 26, 2023 17:09


          Enum8 and Enum16 supported

8488f9a


          flake8 fixed

aa3a31f


          more flake8 fixes

0c3349b

coveralls commented Sep 28, 2023

coverage: 85.9% (+0.005%) from 85.895% when pulling 0c3349b on akurdyukov:master into 3ce18ad on xzkostyan:master.

xzkostyan merged commit 1c7553d into xzkostyan:master

27 of 48 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment