Make nodegit context-aware for compatibility with Electron 9 and beyond

未关闭
#1,774 27 条评论 6 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
4/5
预计耗时
3-5 天
新手友好度
35/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
停滞
技术栈
electron, javascript, node.js
领域
desktop

调研方向

从 Electron 9 的破坏性变更文档和相关 issue #1298 开始,然后检查 nodegit 的原生 Node 绑定以及已报告的 Electron 兼容性行为。当 nodegit 能够在启用 Electron renderer process 重用的情况下正常工作时,即视为完成,包括那些无法再禁用非 context-aware 原生模块的版本。

由索引模型根据 Issue 内容生成。

描述

Electron 9 changed the default value of app.allowRendererProcessReuse to true, which breaks nodegit. Setting it to false manually fixes the issue, but this will only be possible until Electron 11.

Are there any plans to fix this? Seems like #1298 might be related. Also see https://github.com/electron/electron/issues/18397 for more information.

System information

  • node version: v12.16.3
  • npm or yarn version: 6.14.4
  • OS/version/architecture: win32 10.0.18363 x64
  • Applicable nodegit version: 0.26.5
node -v
npm -v # (or yarn -v)
node -e "console.log(process.platform)"
node -e "console.log(require('os').release())"
node -e "console.log(console.log(process.arch))"
主要语言
JavaScript
星标
5.8k
派生
704
PR 合并指标
30 天内没有已合并 PR

贡献指南

打开贡献指南

从这里开始

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

nodegit/nodegit 的其他 Issue

查看 nodegit/nodegit 的全部 Issue

相似的 Issue

更多 JavaScript Issue

把新 issue 发到你的邮箱

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