jamesplease/react-request

UglifyJS fails to minify library files

Open

#174 aberto em 25 de abr. de 2018

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)JavaScript (21 forks)github user discovery
buildhelp wanted

Métricas do repositório

Stars
 (358 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

I enjoy using react-request in my project, thanks for creating it!

I decided to try the new version 3 today, but UglifyJS fails to minify it because of the let keywords in the CommonJS version of the library.

I think it happens because the babel-env plugin configuration is missing the targets section and newer language features are not transpiled.

I am going to amend my configuration to transpile the new react-request version, but it would be nice to have it transpiled out of the box. Moreover, the majority of bundler configuration docs and tutorials assume the code in node_modules is already transpiled to ES5.

uglifyjs-error uglifyjs-error-2

Guia do colaborador

Direção de pesquisa
Inspecione a configuração do Babel no package.json ou .babelrc e garanta que o campo 'targets' esteja definido para transpilar para ES5, por exemplo, usando 'browserslist' com 'last 1 version' ou 'ie 11'. Verifique também se a saída CommonJS da biblioteca está sendo processada corretamente pelo babel.
Pilha de tecnologia
javascriptreact
Domain
frontendbuild system
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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
JavaScriptNode.jsnpm
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65