jaredpalmer/after.js

Razzle as dev dependency?

Ouverte

#419 ouverte le 10 janv. 2021

Voir sur GitHubGuide contributeur
 (5 commentaires) (4 réactions) (0 personne assignée)TypeScript (195 forks)github user discovery
after.jsgood first issue

Métriques du dépôt

Stars
 (4 090 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

🚀 Feature request

All Razzle examples use Razzle package as dev dependency. Except AfterJS - it needs Razzle as regular dependency because of razzle-dev-utils.

Current Behavior

If Razzle stays as a dev dep, I get this error message in production build:

Error: Cannot find module 'razzle-dev-utils/logger'
 Require stack:
 -  /app/node_modules/@jaredpalmer/after/build/cjs/getAssets.js
 - /app/node_modules/@jaredpalmer/after/build/cjs/render.js
 - /app/node_modules/@jaredpalmer/after/build/cjs/index.js

Desired Behavior

I would like to add AfterJS to my Razzle based project without moving Razzle from dev to regular dependencies.

Suggested Solution

Is using razzle-dev-utils in AfterJS worth? Can it be replaced with something?

Guide contributeur

Direction de recherche
Examinez si la dépendance à razzle dev utils dans AfterJS peut être remplacée par une alternative qui ne nécessite pas de déplacer Razzle des dépendances de développement aux dépendances régulières. Étudiez comment razzle dev utils est utilisé et si sa fonctionnalité peut être reproduite ou obtenue à partir d'autres sources.
Stack technique
typescriptreactnodejs
Domaine
frontendbackend
Type d'issue
Fonctionnalité
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.
Une demi journée
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.jsnpm or yarnReact
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65