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

Offen
#8,016 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
58/100
Issue-Typ
Refactoring
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
javascript, node.js

Rechercherichtung

Beginne mit dem root package.json und dem stackgl_modules-Teilprojekt, überprüfe anschließend die Änderungen aus #6004 und die in #7862 genannte Alternative. Verfolge die Skripte, die auf das Teilprojekt verweisen, und führe die vorhandenen Build- und Test-Befehle aus. Als erledigt gilt, dass Abhängigkeiten vom Root aus verwaltet werden und die Skripte ohne die frühere Node/npm-Kompatibilitätsbeschränkung funktionieren.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

chore P3 plotly-internal size: 1 task
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
Vorherrschende Sprache
JavaScript
Sterne
18.3k
Forks
2k
Ø Merge
2 T. 12 Std.
Gemergte PRs (30 T.)
28

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus plotly/plotly.js

Alle Issues in plotly/plotly.js

Ähnliche Issues

Weitere Issues zu JavaScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.