createInterface indents a copied type by its source column
维护者通常 1 天内回复
已经有一个关联 PR 被合并了。
- #9144 来自 @cknitt —— 已合并
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 74/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- ocaml
- 领域
- compilers
调研方向
从 analysis/src/create_interface.ml 开始，重点查看 Source_file_extractor.extract 和 process_signature 中的 Sig_type 分支，issue在此指出了复制文本和缩进处理。使用提供的 Make 示例重现问题，并检查像 module M = {type t = int} 这样的一行 module 是否也有相同问题。完成标准是生成的接口在输出位置缩进复制的类型声明；payload没有指定测试文件。
由索引模型根据 Issue 内容生成。
描述
createInterface copies a type declaration from the source with the column it has there as its indentation, instead of the indentation of the place it is printed at.
module Make = (X: {type t}) => {
type u = X.t
}
generates
module Make: (
X: {
type t
},
) =>
{
type u = X.t
}
type t is at column 19 in the source. Source_file_extractor.extract (analysis/src/create_interface.ml) prefixes the first line of the copied text with col_start spaces, and the Sig_type case of process_signature adds that text without its indent. A type in a module written on one line, such as module M = {type t = int}, is likely affected the same way.
- 主要语言
- OCaml
- 星标
- 7.5k
- 派生
- 484
- 平均合并
- 20 小时 43 分钟
- 30 天内合并 PR
- 263
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
rescript-lang/rescript 的其他 Issue
-
难度 1/5 1 小时以内 新手友好度 85/100
rescript-lang/rescript#9139 ·
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 35/100
rescript-lang/rescript#9151 ·
维护者通常 1 天内回复
-
Printed @as names of fields and constructors turn non-ASCII letters into other characters可能已有人在做 @cknitt 于 1 天前认领。 未关闭
难度 4/5 3-5 天 新手友好度 25/100
rescript-lang/rescript#9142 ·
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 35/100
rescript-lang/rescript#9130 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 4/5 1-2 天 新手友好度 55/100
rescript-lang/rescript#9122 ·
维护者通常 1 天内回复
查看 rescript-lang/rescript 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 85/100
维护者通常 1 天内回复
-
JS/TS: JSX props compile errors should link to a doc page explaining the static-list requirement未关闭area: js-runtime enhancement exists-upstream interop priority: low
难度 2/5 1-3 小时 新手友好度 78/100
shayanhabibi/fickl#307 ·
维护者通常 1 天内回复
-
bug fmt low priority
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
bot-triaged module: inductor oncall: pt2 topic: fuzzer triaged
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
bug needs-triage
难度 2/5 1-3 小时 新手友好度 75/100
microsoft/DirectXShaderCompiler#9087 ·
维护者通常 1 天内回复