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.

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

Upgrade miniflare to 2.0.0-rc.4 & pages dev quick wins #138

Merged
merged 3 commits into from
Dec 19, 2021
Merged

Upgrade miniflare to 2.0.0-rc.4 & pages dev quick wins #138

merged 3 commits into from
Dec 19, 2021

Conversation

mrbbot
Copy link
Contributor

@mrbbot mrbbot commented Dec 19, 2021

Hey! 👋 This PR upgrades miniflare to version 2.0.0-rc.4: https://github.com/cloudflare/miniflare/releases/tag/v2.0.0-rc.4.

This version includes support for logging unhandled rejections instead of crashing, which is automatically enabled when using the CLI, and also available as the logUnhandledRejections option. Whilst adding this, I thought I'd make a couple other tiny changes to pages dev, which have been approved by @GregBrimble in conversation.

  • Enabled source map support for nicer stack traces (requires adding sourcemap: true to the esbuild options in @cloudflare/pages-functions-compiler buildWorker function)
  • Enabled incoming HTTP request logging
  • Added colours and prefixes to Miniflare logs (this also dims internal lines in stack traces for easier debugging)

Logging unhandled rejections and source mapping are process wide, but since this is in the pages dev command handler, it shouldn't be a problem.

unknown

- Enabled source map support
- Log unhandled rejections instead of crashing
- Add colours and prefixes to logs
@changeset-bot
Copy link

changeset-bot bot commented Dec 19, 2021

🦋 Changeset detected

Latest commit: eddb2da

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
wrangler Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

@threepointone threepointone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you remove react-error-boundary by mistake?

packages/wrangler/package.json Show resolved Hide resolved
Copy link
Contributor

@threepointone threepointone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving this, will wait for @GregBrimble to have a look before merging. Thanks @mrbbot!

@threepointone
Copy link
Contributor

Nvm, I'm going to just merge this and take for a spin, that's what alphas are for :)

@threepointone threepointone merged commit 43e7a82 into cloudflare:main Dec 19, 2021
@mrbbot mrbbot deleted the miniflare-2.0.0-rc.4 branch December 22, 2021 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants