docs: API.md still documents Spec::layer_sql/stat_sql removed in #569
维护者通常 2 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 1/5
- 预计耗时
- 1 小时以内
- 新手友好度
- 88/100
- Issue 类型
- 文档
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- rust
调研方向
过时的引用位于 src/doc/API.md 的“Query Introspection Methods”部分,大约在第 268 行:一对用于 layer_sql / stat_sql 的表格行,以及调用 spec.layer_sql(i) / spec.stat_sql(i) 的示例行。在仓库中搜索 layer_sql 和 stat_sql,确认文档中没有其他地方引用它们。完成的标准是:删除这些表格行和示例行,不再有任何引用,并且关于方言层 SQL 生成的可选说明已添加或有意省略。
由索引模型根据 Issue 内容生成。
描述
src/doc/API.md still documents Spec::layer_sql() and Spec::stat_sql(), but these methods were removed in #569 (afe60e64).
The affected section is "Query Introspection Methods" (around line 268): the table lists layer_sql / stat_sql, and the example below it calls spec.layer_sql(i) / spec.stat_sql(i), which no longer compiles against main.
For context: these accessors were already vestigial before removal — since #82, Spec::new() populated the backing fields with vec![None; ...], so they returned None in every released crate. The docs made the dead API look live (this confused at least one downstream binding — posit-dev/ggsql-python — into mirroring it).
Suggested fix: drop the two rows from the introspection table and the corresponding lines from the example. It may also be worth a sentence in that section noting that per-layer SQL is constructed centrally through the dialect layer at execution time and is no longer exposed on Spec.
- 主要语言
- Rust
- 星标
- 565
- 派生
- 31
- 平均合并
- 2 天 12 小时
- 30 天内合并 PR
- 16
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
posit-dev/ggsql 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 68/100
posit-dev/ggsql#565 · 2 个 reaction ·
维护者通常 2 天内回复
-
Update the positron dep for the extension to 0.2.9可能已有人在做 @thomasp85 于 17 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 2 天内回复
-
writer
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 2 天内回复
-
plot building
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 2 天内回复
-
bug plot building
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 2 天内回复
相似的 Issue
-
priority: low
难度 2/5 1-3 小时 新手友好度 65/100
storytold/photocraft#6286 · 1 条评论 ·
维护者通常 1 天内回复
-
Four problem loader browser tests fail on Windows可能已有人在做 @Patriciaath999 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 88/100
维护者通常 1 天内回复
-
original-bug
难度 2/5 1-3 小时 新手友好度 62/100
维护者通常 1 天内回复
-
effort/S kind/docs
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
starkware-libs/sequencer#15229 ·
维护者通常 1 天内回复