bug: `@stencil/core` should be a devDependency

Ouverte Adaptée aux débutants
#1,519 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
73/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
typescript
Domaine
build-system

Piste de recherche

Commencez par lire package.json et stencil.config.js, en particulier le paramètre externalRuntime mentionné dans l’issue. Vérifiez si @stencil/core figure dans les dépendances d’exécution du paquet publié, puis vérifiez les métadonnées du paquet après son déplacement vers les dépendances de développement. Le travail est terminé lorsque npm install ionicons n’installe plus @stencil/core.

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

Description

triage
Current Behavior

@stencil/core is a dependency of this project, npm install ionicons installs @stencil/core.

Expected Behavior

@stencil/core is a devDependency of this project, npm install ionicons does not install @stencil/core.

Steps to Reproduce

npm install ionicons

Code Reproduction URL

No response

Additional Information

Hi, thanks a lot for the icons 🐱

The web component is generated with externalRuntime: false in stencil.config.js (since commit e04eccc8683289c0e3a5be4bccec7e7a3e9b5c52). In my understanding, this would mean that @stencil/core is actually not needed at runtime and can be declared as a devDependency of the ionicons package.

A slim (or in this case empty) dependency tree would be much appreciated for my use case.

Langage dominant
TypeScript
Étoiles
18.2k
Forks
2k
Métriques de merge des PR
Aucune PR mergée en 30 j

Préparer son environnement

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.

Autres issues de ionic-team/ionicons

Toutes les issues de ionic-team/ionicons

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

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