-
Notifications
You must be signed in to change notification settings - Fork 979
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
Add Astro web framework support #5527
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #5527 +/- ##
==========================================
+ Coverage 55.71% 55.78% +0.06%
==========================================
Files 322 324 +2
Lines 21836 21892 +56
Branches 4454 4462 +8
==========================================
+ Hits 12166 12212 +46
- Misses 8602 8606 +4
- Partials 1068 1074 +6
... and 1 file with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed a couple nits, added test coverage. Thanks!
Co-authored-by: James Daniels <jamesdaniels@google.com>
Co-authored-by: James Daniels <jamesdaniels@google.com>
Description
In conjunction with FirebaseExtended/firebase-framework-tools#61, this PR adds Astro support to Firebase web frameworks.
A temp change was made inframeworks/index.ts
to support ESM, but another PR (#5540) is open that should fix this. That PR should be merged before this one.Scenarios Tested
Tested with the Astro skeleton template, including static site generation, API routes, and partial hydration with a Svelte component.