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(nextjs): Stop excluding withSentryConfig from serverless bundles #6267

Merged
merged 8 commits into from Nov 23, 2022
Merged

fix(nextjs): Stop excluding withSentryConfig from serverless bundles #6267

merged 8 commits into from Nov 23, 2022
+105 −35

Commits on Nov 23, 2022

  1. extract main logic in withSentryConfig into a helper function

    @lobsterkatie
    lobsterkatie committed Nov 23, 2022
    Copy the full SHA
    4b40f61 View commit details
    Browse the repository at this point in the history

  2. only require webpack.ts at build time

    @lobsterkatie
    lobsterkatie committed Nov 23, 2022
    Copy the full SHA
    650cd95 View commit details
    Browse the repository at this point in the history

  3. add webpack.ts to rollup config

    @lobsterkatie
    lobsterkatie committed Nov 23, 2022
    Copy the full SHA
    7e1b8bc View commit details
    Browse the repository at this point in the history

  4. exclude webpack.ts but not withSentryConfig.ts

    @lobsterkatie
    lobsterkatie committed Nov 23, 2022
    Copy the full SHA
    5f9a08a View commit details
    Browse the repository at this point in the history

  5. add flag noting that webpack.ts has been loaded

    @lobsterkatie
    lobsterkatie committed Nov 23, 2022
    Copy the full SHA
    eda94f7 View commit details
    Browse the repository at this point in the history

  6. rename index.test.ts to withSentryConfig.test.ts

    @lobsterkatie
    lobsterkatie committed Nov 23, 2022
    Copy the full SHA
    bf47570 View commit details
    Browse the repository at this point in the history

  7. add and fix tests

    @lobsterkatie
    lobsterkatie committed Nov 23, 2022
    Copy the full SHA
    a9af4aa View commit details
    Browse the repository at this point in the history

  8. Use global instead of process.env

    @lforst
    lforst committed Nov 23, 2022
    Copy the full SHA
    728562b View commit details
    Browse the repository at this point in the history