jaredpalmer/after.js

Razzle as dev dependency?

Aberta

#419 aberto em 10 de jan. de 2021

Ver no GitHubGuia do colaborador
 (5 comentários) (4 reações) (0 responsável)TypeScript (195 forks)github user discovery
after.jsgood first issue

Métricas do repositório

Stars
 (4.090 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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?

Guia do colaborador

Direção de pesquisa
Investigue se a dependência do razzle dev utils no AfterJS pode ser substituída por uma alternativa que não exija mover o Razzle de dependências de desenvolvimento para dependências regulares. Analise como o razzle dev utils é usado e se sua funcionalidade pode ser replicada ou obtida de outra fonte.
Pilha de tecnologia
typescriptreactnodejs
Domain
frontendbackend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Node.jsnpm or yarnReact
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65