plotly / plotly/plotly.js

[CHORE]: Reintegrate `stackgl_modules` into top level library

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

还没有人认领这个 Issue。

chore P3 plotly-internal size: 1 task
主要语言
JavaScript
星标
18.3k
派生
2k
平均合并
2 天 12 小时
30 天内合并 PR
28

描述

Description

Reintegrate the stackgl_modules sub-project into the top level library.

Why should this change be made?

This was originally moved to a subfolder in #6004 to allow for supporting older versions of Node and npm. That support is no longer required and moving this back would allow us to simplify the repo a bit.

Scope
  • Move dependencies into the root package.json
  • Update scripts to handle new location
Notes
  • It's possible that it would still be beneficial to leave this folder as is since it segregates some dev dependencies that most people don't need
  • This was mentioned as a potential option in #7862

贡献指南

打开贡献指南

从这里开始

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

调研方向

从 root package.json 和 stackgl_modules 子项目开始,然后检查 #6004 中的更改以及 #7862 中提到的替代方案。跟踪引用该子项目的脚本,并运行现有的 build 和 test 命令。完成的标准是从 root 管理依赖项,并且脚本无需受之前的 Node/npm 兼容性限制即可运行。

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

评估

技术栈
javascript, node.js
领域
build-system, tooling
Issue 类型
重构
难度
4/5
预计耗时
3-5 天
活跃度
活跃
描述清晰度
基本清楚
新手友好度
58/100

把新 issue 发到你的邮箱

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