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
  • !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 6 years ago
  • 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.)

Activity

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
  1. 29 Jul, 2019 2 commits
    • Sean Larkin's avatar
      Merge branch 'master' into feature/organize-config · 5ff14aed
      Sean Larkin authored 6 years ago
      5ff14aed
    • Sean Larkin's avatar
      Reorganize webpack configuration structure · 94f91ab7
      Sean Larkin authored 6 years ago
      94f91ab7
0 Assignees
None
Assign to
0 Reviewers
Request review from
Labels
0
None
0
None
    Assign labels
  • Manage project labels

Milestone
No milestone
None
None
Time tracking
Lock merge request
Unlocked
participants
Reference:
Source branch: github/fork/TheLarkInn/feature/organize-config

Menu

Explore Projects Groups Snippets