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

refactor(prerender): build with esbuild and introduce conditional exports #11337

Merged
merged 3 commits into from
Aug 22, 2024

Conversation

Josh-Walker-GM
Copy link
Collaborator

Switches the prerender package to be built with esbuild rather than babel.

This change doesn't not use module or moduleResolution Node16 in the build version of the tsconfig. This is because it would pick up cjs/esm issues with apollo and vite. I will look to follow up with fixes that can enable us to use the modern settings in the build version also.

@Josh-Walker-GM Josh-Walker-GM added the release:breaking This PR is a breaking change label Aug 22, 2024
@Josh-Walker-GM Josh-Walker-GM added this to the next-release-major milestone Aug 22, 2024
@Josh-Walker-GM Josh-Walker-GM self-assigned this Aug 22, 2024
@Josh-Walker-GM Josh-Walker-GM marked this pull request as ready for review August 22, 2024 01:14
@Josh-Walker-GM Josh-Walker-GM merged commit b544605 into main Aug 22, 2024
47 checks passed
@Josh-Walker-GM Josh-Walker-GM deleted the jgmw/chore-build-prerender branch August 22, 2024 01:31
dac09 added a commit to dac09/redwood that referenced this pull request Aug 22, 2024
…into feat/rw-uploads-extension

* 'feat/rw-uploads-extension' of github.com:dac09/redwood:
  chore(test-fixture): correct rebuild script string suffix substitution  (redwoodjs#11339)
  refactor(forms): Build with esbuild and add conditional exports (redwoodjs#11338)
  refactor(prerender): build with esbuild and introduce conditional exports (redwoodjs#11337)
  Adds more tests! Refactor rw-jobs-worker  (redwoodjs#11335)
  refactor(auth-providers): switch to esbuild for some 'web' packages (redwoodjs#11334)
  Jobs doc updates (redwoodjs#11333)
  chore(ci): Split formatting check into separate job  (redwoodjs#11332)
  Updates caching doc code snippets to be consistent (redwoodjs#11331)
  chore(tsconfig): Properly link to api-server's tsconfig (redwoodjs#11330)
  Adds background job scheduling and execution (redwoodjs#11238)
  [gh11317] Added documentation for working with Jest and Alias Paths (redwoodjs#11323)
  fix(record): Update dependencies and build dual esm/cjs (redwoodjs#11300)
  chore(build): Consistent tsconfig and buildinfo naming for CJS (redwoodjs#11326)
  chore(test): Fix RSC server cell smoke test (redwoodjs#11327)
  chore(playwright): Make RSC tests less flakey (redwoodjs#11325)
  chore(ci): Temporary fix for format checking on all PRs (redwoodjs#11324)
  chore(build): Move more repetitive code into framework-tools (redwoodjs#11320)
  chore(docs): fix docs formatting (redwoodjs#11321)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:breaking This PR is a breaking change
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant