plotly / plotly/plotly.py

Add a script to simplify testing plotly.js commits on the plotly.py side

未关闭
#4,638 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

feature P3 testing
主要语言
Python
星标
18.8k
派生
2.8k
平均合并
16 小时 26 分钟
30 天内合并 PR
21

描述

In our contributing docs we have a section that describes how one could use a plotly.js bundle to test bundles from the last commits of pull requests or any plotly.js branch.
https://github.com/plotly/plotly.py/blob/master/contributing.md#circleci-release

Screenshot from 2024-06-17 10-01-10

This is generally fine; however, it has certain limitations.
For example the CircleCI keeps the artifatcs for just for few days (around two weeks).
So one may need to rerun the build process on CI after a while.
etc.


I suggest we add a script where one provides a link to a plotly.js commit similar to https://github.com/plotly/plotly.js#caf32e57cfdbd370076b926f84c114093f62f0b9 and then it updates the package file, install plotlyjs and rebuilt the bundles. Then copy the new file over.
Something like:

cd packages/javascript/jupyterlab-plotly/
npm install https://github.com/plotly/plotly.js#caf32e57cfdbd370076b926f84c114093f62f0b9
cd node_modules/plotly.js
npm install
npm run bundle
cp dist/plotly.min.js ../../../../../packages/python/plotly/plotly/package_data/plotly.min.js

贡献指南

打开贡献指南

从这里开始

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

调研方向

从 contributing.md 中的 CircleCI 发布部分以及现有的包路径 packages/javascript/jupyterlab-plotly/ 开始。检查 issue 中描述的 npm 安装和 bundle 命令,包括目标路径 packages/python/plotly/plotly/package_data/plotly.min.js;完成的标准是脚本能够接受一个 plotly.js 提交链接,重新构建 bundle,并复制生成的文件。

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

评估

技术栈
javascript, python, shell
领域
build-system, tooling
Issue 类型
功能
难度
3/5
预计耗时
1-2 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 发到你的邮箱

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