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
  • !6295

Disable Terser function inlining

  • Review changes

  • Download
  • Email patches
  • Plain diff
Administrator requested to merge github/fork/jfo84/fix/disable-function-inlining into master 6 years ago
  • Overview 8
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: jfo84

There's a bug in a library I contribute to that is causing us a lot of trouble. There's plenty of documentation of the bug, and we can reliably reproduce it in our production app. Can you disable this behavior here to help out one of our users?

Loading
Loading

Activity


  • Administrator added CLA Signed label 6 years ago

    added CLA Signed label

  • Administrator
    Administrator @root · 6 years ago
    Author Contributor

    Created by: Downchuck

    The link on Terser JS shows the issue as being fixed -- have you sent them a note to investigate as well?

  • Administrator
    Administrator @root · 6 years ago
    Author Contributor

    Created by: jfo84

    @Downchuck Which link? There's an open issue on Terser that you can find in the linked issue

  • Administrator
    Administrator @root · 6 years ago
    Author Contributor

    Created by: Downchuck

    @jfo84 My mistake - I saw a note already in the CRA code to a closed issue https://github.com/terser-js/terser/issues/120 - In your post you're working a different issue https://github.com/terser-js/terser/issues/244

  • Administrator
    Administrator @root · 6 years ago
    Author Contributor

    Created by: jfo84

    @Downchuck You are a bull in a china shop, my friend

  • Administrator
    Administrator @root · 6 years ago
    Author Contributor

    Created by: Downchuck

    @jfo84 If you're gonna make an omelette, you gotta break a few dishes.

  • Administrator
    Administrator @root · 6 years ago
    Author Contributor

    Created by: jfo84

    @Downchuck I respect that

  • Administrator
    Administrator @root · 6 years ago
    Author Contributor

    Created by: stale[bot]

    This pull request has been automatically marked as stale because it has not had any recent activity. It will be closed in 5 days if no further activity occurs.

  • Administrator added stale label 6 years ago

    added stale label

  • Administrator
    Administrator @root · 6 years ago
    Author Contributor

    Created by: stale[bot]

    This pull request has been automatically closed because it has not had any recent activity. If you have a question or comment, please open a new issue. Thank you for your contribution!

  • Administrator closed 6 years ago

    closed

Please register or sign in to reply
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/jfo84/fix/disable-function-inlining