parcel-bundler/parcel

Different error message for disabled autoinstall

Ouverte

#6 165 ouverte le 22 avr. 2021

Voir sur GitHubGuide contributeur
 (11 commentaires) (1 réaction) (0 personne assignée)JavaScript (2 274 forks)batch import
:bug: BugAutoinstallGood First Issue✨ Parcel 2

Métriques du dépôt

Stars
 (44 030 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

🐛 bug report

At the moment, the error message for parcel build --no-autoinstall and

  • a missing parcel plugin specified in the config that usually would get autoinstalled (e.g. the SASS transformer) or
  • packageManager.resolve and packageManager.require

is just Cannot find module foo, did you mean bar.

In this case, it should say something along the lines of Autoinstall is disabled, please install foo yourself.

Relevant code

https://github.com/parcel-bundler/parcel/blob/3baf990a686d1c1dd0c5f4360296de890cc00c47/packages/core/core/src/loadParcelPlugin.js#L82-L126

Guide contributeur

Direction de recherche
Examinez le fichier `packages/core/core/src/loadParcelPlugin.js` autour des lignes 82-126 où le message d'erreur est généré. L'objectif est de détecter quand l'auto installation est désactivée et de fournir un message différent. Vérifiez s'il existe un indicateur ou une configuration indiquant l'état de l'auto installation. Implémentez une vérification conditionnelle pour afficher un message comme 'L'auto installation est désactivée, veuillez installer foo vous même.' Assurez vous que le comportement existant reste intact lorsque l'auto installation est activée.
Stack technique
javascriptnodejs
Domaine
build systemcli
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Node.jsParcel plugin system
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75