plotly / plotly/plotly.js

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

Ouverte
#8,016 0 commentaires 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

chore P3 plotly-internal size: 1 task
Langage dominant
JavaScript
Étoiles
18.3k
Forks
2k
Merge moyen
2 j 12 h
PR mergées (30 j)
28

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

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par le root package.json et le sous-projet stackgl_modules, puis examinez les changements de #6004 et l’alternative mentionnée dans #7862. Suivez les scripts qui font référence au sous-projet et exécutez les commandes de build et de test existantes. Le travail est terminé lorsque les dépendances sont gérées depuis le root et que les scripts fonctionnent sans l’ancienne contrainte de compatibilité de Node/npm.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript, node.js
Domaine
build-system, tooling
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
Active
Clarté
Plutôt claire
Accessibilité débutants
58/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.