[CHORE]: Reintegrate `stackgl_modules` into top level library
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 58/100
- Issue type
- Refactor
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- javascript, node.js
- Domain
- build-system, tooling
Research direction
Start with the root package.json and the stackgl_modules sub-project, then review the changes from #6004 and the alternative noted in #7862. Trace the scripts that reference the sub-project and run the existing build and test commands. Done means dependencies are handled from the root and the scripts work without the former Node/npm compatibility constraint.
Written by the indexing model from the issue text.
Description
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
- Dominant language
- JavaScript
- Stars
- 18.3k
- Forks
- 2k
- Avg merge
- 2d 17h
- Merged PRs (30d)
- 22
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from plotly/plotly.js
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 85/100
-
chore P3 plotly-internal size: 3 task
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
-
chore P1 plotly-internal size: 1 task
Difficulty 1/5 Under an hour Newbie friendliness 82/100
-
chore P3 plotly-internal size: 1 task
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
All issues in plotly/plotly.js
Similar issues
-
Difficulty 1/5 1-3 hours Newbie friendliness 78/100
Mintplex-Labs/anything-llm#6490 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
-
[quality] workflow-scripts guard reads test:unit:coverage as skipping the unit suite, blocking #632 Openagent/quality hive/hosted-available-lke648397-260827-5n31 quality testing
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
area-clientside-dartpad
Difficulty 2/5 1-3 hours Newbie friendliness 62/100