Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(gatsby): move require out of hot function #36253

Merged
merged 5 commits into from
Jul 28, 2022
Merged

fix(gatsby): move require out of hot function #36253

merged 5 commits into from
Jul 28, 2022

Commits on Jul 27, 2022

  1. fix(gatsby): move require out of hot function 

    This require showed up in a performance profile as taking ~600ms during a large query as `resolveField` is called a lot. We can require it out of the function to avoid this.
    @KyleAMathews
    KyleAMathews authored Jul 27, 2022
    Configuration menu
    Copy the full SHA
    b2eef68 View commit details
    Browse the repository at this point in the history

  2. not sure why it needs the .default here

    @KyleAMathews
    KyleAMathews authored Jul 27, 2022
    Configuration menu
    Copy the full SHA
    a24fc9a View commit details
    Browse the repository at this point in the history

  3. Update node-model.js

    @KyleAMathews
    KyleAMathews authored Jul 27, 2022
    Configuration menu
    Copy the full SHA
    63816f2 View commit details
    Browse the repository at this point in the history

  4. Update node-model.js

    @KyleAMathews
    KyleAMathews authored Jul 27, 2022
    Configuration menu
    Copy the full SHA
    25fab53 View commit details
    Browse the repository at this point in the history

  5. Fix lint

    @KyleAMathews
    KyleAMathews committed Jul 27, 2022
    Configuration menu
    Copy the full SHA
    1886624 View commit details
    Browse the repository at this point in the history