Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

docs: API.md still documents Spec::layer_sql/stat_sql removed in #569

未关闭 适合新手
#590 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 2 天内回复

还没有人认领这个 Issue。

评估

难度
1/5
预计耗时
1 小时以内
新手友好度
88/100
Issue 类型
文档
描述清晰度
描述清楚
活跃度
活跃
技术栈
rust
领域
documentation

调研方向

过时的引用位于 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,通用步骤见我们的新手贡献指南。

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

posit-dev/ggsql 的其他 Issue

查看 posit-dev/ggsql 的全部 Issue

相似的 Issue

更多 Rust Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。