jamesplease/react-request

UglifyJS fails to minify library files

Open

#174 geöffnet am 25. Apr. 2018

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (21 Forks)github user discovery
buildhelp wanted

Repository-Metriken

Stars
 (358 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Überprüfe die Babel Konfiguration in package.json oder .babelrc und stelle sicher, dass das Feld 'targets' auf ES5 Transpilation gesetzt ist, z.B. mit 'browserslist' und 'last 1 version' oder 'ie 11'. Prüfe auch, ob die CommonJS Ausgabe der Bibliothek korrekt von Babel verarbeitet wird.
Tech Stack
javascriptreact
Domain
frontendbuild system
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptNode.jsnpm
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65