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

Add ColumnOperators.hash (fixes #262) #263

Merged

CaselIT merged 1 commit into sqlalchemy:main from mrcljx:patch-1

Dec 30, 2023

Contributor

mrcljx commented Dec 30, 2023

Description

Added __hash__ like it's done in SQLAlchemy 2.0.

Checklist

This pull request is:

A short code fix
- please include the issue number, and create an issue if none exists, which
  must include a complete example of the issue. one line code fixes without an
  issue and demonstration will not be accepted.
- Please include: Fixes: #<issue number> in the commit message
- please include tests. one line code fixes without tests will not be accepted.


          Add ColumnOperators.__hash__ (fixes sqlalchemy#262)

5c3ba15

Contributor Author

mrcljx commented Dec 30, 2023

Please provide guidance on how to add a test here.

Member

CaselIT commented Dec 30, 2023

Thanks, it's fine as is, since this library is mostly legacy at this point.

Will release after the merge

CaselIT merged commit d71cd0e into sqlalchemy:main

2 checks passed

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