required should be inferred from the return type of a hybrid property
还没有人认领这个 Issue。
评估
调研方向
首先跟踪 hybrid_property 字段如何转换为 GraphQL 字段，然后将该路径与现有的 column property 可空性推断进行比较。当声明的 bool 返回类型生成必需的 Boolean 字段，而 bool | None 生成可选字段，并且两个情况都有覆盖时，修改即完成。
由索引模型根据 Issue 内容生成。
描述
With a definition like:
@hybrid_property
def is_active(self) -> bool:
return self.state == 'ACTIVE'
The resulting graphql field type is currently marked Boolean unless overridden by ORMField(). Column properties correctly determine the nullability from the column definition. It seems reasonable to do the same for the hybrid_property based on the declared type. If the return type were bool | None, then it would be optional, otherwise it should be required.
- 主要语言
- Python
- 星标
- 985
- 派生
- 224
- PR 合并指标
- 30 天内没有已合并 PR
贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
graphql-python/graphene-sqlalchemy 的其他 Issue
-
难度 4/5 3-5 天 新手友好度 38/100
-
难度 5/5 一周以上 新手友好度 35/100
-
难度 5/5 一周以上 新手友好度 25/100
graphql-python/graphene-sqlalchemy#422 · 1 条评论 ·
-
难度 5/5 一周以上 新手友好度 25/100
graphql-python/graphene-sqlalchemy#419 · 3 条评论 ·
-
难度 5/5 一周以上 新手友好度 25/100
graphql-python/graphene-sqlalchemy#418 · 2 条评论 · 2 个 reaction ·
查看 graphql-python/graphene-sqlalchemy 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 75/100
anthropics/skills#1811 · 1 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 75/100
speaches-ai/speaches#678 ·
-
bug
难度 2/5 1-3 小时 新手友好度 75/100
datalayer/mcp-compose#42 ·
-
难度 2/5 1-3 小时 新手友好度 75/100
conda-forge/spacy-feedstock#177 ·
-
难度 2/5 1-3 小时 新手友好度 70/100
UKGovernmentBEIS/inspect_evals#2523 ·