getReferences Callback is required
未关闭
还没有人认领这个 Issue。
- 主要语言
- JavaScript
- 星标
- 5.8k
- 派生
- 704
- PR 合并指标
- 30 天内没有已合并 PR
描述
System information
- node version:
- npm or yarn version:
- OS/version/architecture:
- Applicable nodegit version:
v12.12.0
6.11.3
darwin
19.0.0
x64
This seems like a bug:
import * as Git from "nodegit"
Git.Repository.open(".")
.then(async repo => {
return repo.getReferences(Git.Reference.TYPE.OID)
})
.catch(console.error)
❯ yarn -s ts-node test.ts
Error: Callback is required and must be a Function.
at /Users/jasonkuhrt/projects/prisma-labs/libre/node_modules/nodegit-promise/lib/node-extensions.js:24:20
at tryCallTwo (/Users/jasonkuhrt/projects/prisma-labs/libre/node_modules/nodegit-promise/lib/core.js:45:5)
at doResolve (/Users/jasonkuhrt/projects/prisma-labs/libre/node_modules/nodegit-promise/lib/core.js:173:13)
at new Promise (/Users/jasonkuhrt/projects/prisma-labs/libre/node_modules/nodegit-promise/lib/core.js:65:3)
at Repository.getReferences (/Users/jasonkuhrt/projects/prisma-labs/libre/node_modules/nodegit-promise/lib/node-extensions.js:19:12)
at /Users/jasonkuhrt/projects/prisma-labs/libre/test.ts:6:17
at Generator.next (<anonymous>)
at /Users/jasonkuhrt/projects/prisma-labs/libre/test.ts:8:71
at new Promise (<anonymous>)
at __awaiter (/Users/jasonkuhrt/projects/prisma-labs/libre/test.ts:4:12)
Maybe related https://github.com/nodegit/nodegit/issues/1726
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
将 test.ts 中报告的调用与 node-extensions.js 中的 Repository.getReferences 进行比较,并查看相关 issue #1726。复现所示的 Node/TypeScript 示例,并检查现有的 promise/callback 行为;完成的标准是该调用不再引发所报告的 callback 错误,并且可以观察到其 references 结果。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- git, javascript, node.js
- 领域
- api, backend
- Issue 类型
- 缺陷
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 35/100