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
  • !7860
An error occurred while fetching the assigned milestone of the selected merge_request.

Mark TypeScript as an optional peer dependency for react-scripts

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/dstaley/patch-1 into master 5 years ago
  • Overview 6
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: dstaley

This PR adds TypeScript as an optional peer dependency for react-scripts. This modification is necessary to get react-scripts working with Yarn v2, and will enable the TypeScript template for create-react-app to work with Yarn v2.

This PR uses the peerDependenciesMeta key, which signifies to Yarn that typescript is an optional dependency. npm will support the key in the future, in which case "typescript": "*" can be added to peerDependencies. Until that change is made, simply having peerDependenciesMeta will be enough for Yarn, and won't adversely impact users of npm.

Other relevant issues: yarnpkg/berry/pull/475

/cc @iansu

Approval is optional

Merged by (Jan 12, 2025 4:48pm UTC)

Loading

Activity


  • Administrator requested review from @root 5 years ago

    requested review from @root

  • Administrator requested review from @root 5 years ago

    requested review from @root

  • Administrator requested review from @root 5 years ago

    requested review from @root

  • Administrator requested review from @root 5 years ago

    requested review from @root

  • Administrator
    Administrator @root · 5 years ago
    Author Contributor

    Created by: facebook-github-bot

    Hi dstaley! Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file.In order for us to review and merge your code, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.

    If you have received this in error or have any questions, please contact us at cla@fb.com. Thanks!

  • Administrator
    Administrator @root · 5 years ago
    Author Contributor

    Created by: iansu

    @dstaley Awesome! I suppose we'll have to publish this change to get the tests passing in Yarn v2? I can merge this right away but it might not be published for a week or two as we're currently working on the next release.

  • Administrator
    Administrator @root · 5 years ago
    Author Contributor

    Created by: dstaley

    @iansu yeah, we'd need npx create-react-app to result in the new version being installed for the tests. It's okay if it waits for the next release! I can setup a reminder to check every couple of days to see if the new release has been published.

  • Administrator added CLA Signed label 5 years ago

    added CLA Signed label

  • Administrator
    Administrator @root · 5 years ago
    Author Contributor

    Created by: facebook-github-bot

    Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

  • Administrator added tag: bug fix label 5 years ago

    added tag: bug fix label

  • Administrator changed milestone to %3.3 5 years ago

    changed milestone to %3.3

  • Administrator
    Administrator @root · 5 years ago
    Author Contributor

    Merged by: ianschmitz at 2019-10-31 04:05:30 UTC

  • Administrator closed 5 years ago

    closed

  • Administrator
    Administrator @root · 5 years ago
    Author Contributor

    Created by: ianschmitz

    Thanks @dstaley!

Please register or sign in to reply
0 Assignees
None
Assign to
0 Reviewers
None
Request review from
Labels
0
None
0
None
    Assign labels
  • Manage project labels

Milestone
No milestone
None
None
Time tracking
No estimate or time spent
Lock merge request
Unlocked
1
1 participant
Administrator
Reference:
Source branch: github/fork/dstaley/patch-1

Menu

Explore Projects Groups Snippets