Skip to content
GitLab
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
  • !7450

Reorganize webpack configuration structure

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Administrator requested to merge github/fork/TheLarkInn/feature/organize-config into main Jul 29, 2019
  • Overview 5
  • Commits 2
  • Pipelines 0
  • Changes 6

Created by: TheLarkInn

This PR Kicks off the process of reorganizing the current webpack configuration for CRA. We at webpack want to give first time webpack user's the best possible experience. This demographic includes developers who would eject CRA to experiment, learn, extend possibly for the first time.

We think the current config format has a lot of opportunities for organization. That being said:

Changes in this PR

  • Create development and production config containing their environment specific configuration details.
  • Separate module from the base webpack config and isolate to a separate file for manageability.
  • Add webpack-merge as a dependency to safely compose prod and dev configs with base config
  • Remove some defaulted configuration values already set for specific env's.
  • Add webpack.ProgressPlugin() for development environment
  • Remove chunks:'all' optimization which is only relevant for Multipage App Architectures
  • Any time a CRA Issue was mentioned in comments, I removed prepended commentary (for the sake of readability.)
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/TheLarkInn/feature/organize-config