dropbox / dropbox/sqlalchemy-stubs

Columns should act as proxies to the Comparator class of underlying types

Open
#88 0 comments 0 reactions 0 assignees View on GitHub
enhancement needs discussion topic-plugins topic-stubs
Dominant language
Python
Stars
583
Forks
96
PR merge metrics
No merged PRs in 30d

Description

For example `Column(ARRAY(Integer())).any` is `sqlalchemy.sql.sqltypes.ARRAY.Comparator.any`. We should support this. This requires both a plugin update and filling in comparator classes in stubs.

This may be tricky because column types depend on Python types, not on SQL types, i.e. `Column[str]` not `Column[String]`. Now it looks like choosing the former and not the latter was a bad decision, but now it is hard to change because it may break lots of working annotations in user code.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.