Skip to content
GitLab
    • Explore Projects Groups Snippets
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • C create-react-app
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,547
    • Issues 1,547
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 417
    • Merge requests 417
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Meta
  • create-react-app
  • Merge requests
  • !3743
Something went wrong while fetching comments. Please try again.

Specify JavaScript version for Uglify

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/Timer/restrict-syntax into master 7 years ago
  • Overview 4
  • Commits 1
  • Pipelines 0
  • Changes 2

Created by: Timer

Since this now uses uglify-es under the hood, we want to restrict language support to ES5.

This is in line with the current behavior.

Given we've recently merged browserlist support, we should probably set this option based on specific browsers (in a follow up PR).

Compare
  • master (base)

and
  • latest version
    a3867bc8
    1 commit, 2 years ago

2 files
+ 2
- 1

    Preferences

    File browser
    Compare changes
packages/re‎act-scripts‎
con‎fig‎
webpack.con‎fig.prod.js‎ +1 -0
packag‎e.json‎ +1 -1
packages/react-scripts/config/webpack.config.prod.js
+ 1
- 0
  • View file @ a3867bc8

  • Edit in single-file editor

  • Open in Web IDE


@@ -293,6 +293,7 @@ module.exports = {
// Minify the code.
new UglifyJsPlugin({
uglifyOptions: {
ecma: 5,
compress: {
warnings: false,
// Disabled because of an issue with Uglify breaking seemingly valid code:
packages/react-scripts/package.json
+ 1
- 1
  • View file @ a3867bc8

  • Edit in single-file editor

  • Open in Web IDE


@@ -53,7 +53,7 @@
"react-dev-utils": "^4.2.1",
"style-loader": "0.19.0",
"sw-precache-webpack-plugin": "0.11.4",
"uglifyjs-webpack-plugin": "1.1.4",
"uglifyjs-webpack-plugin": "1.1.6",
"url-loader": "0.6.2",
"webpack": "3.8.1",
"webpack-dev-server": "2.9.4",
0 Assignees
None
Assign to
0 Reviewers
None
Request review from
Labels
1
CLA Signed
1
CLA Signed
    Assign labels
  • Manage project labels

Milestone
2.0
2.0 (expired)
None
Time tracking
No estimate or time spent
Lock merge request
Unlocked
1
1 participant
Administrator
Reference: facebook/create-react-app!3743
Source branch: github/fork/Timer/restrict-syntax

Menu

Explore Projects Groups Snippets