sveltekit-cdk icon indicating copy to clipboard operation
sveltekit-cdk copied to clipboard

Clean up after scope change

Open juranki opened this issue 2 years ago • 0 comments

  • [x] There is now only one adapter, so it should be called adapter here
  • [ ] There's no longer a separate construct for lambda handler, so artifactPath is not needed in RendererProps here
  • [ ] Only CDK v2 is supported and there's just one construct, should it be in @sveltekit-cdk/construct instead of @sveltekit-cdk/constructsv2? eh...
  • [ ] App.render() is now Server.respond(), so RendererProps doesn't fit any more -> ServerProps
  • [ ] SvelteDistribution -> SvelteKitSite
    • Svelte != SvelteKit
    • the construct is a composition of distribution, function and bucket, so distribution is a bad name for it

juranki avatar Mar 13 '22 05:03 juranki