Skip to content

Conversation

@leoortizz
Copy link
Member

@leoortizz leoortizz commented Dec 26, 2023

Description

Fixes #5962.
Closes #6041.

This PR also has 38cc13c from #6643 to have target in build

Scenarios Tested

Sample Commands

yykamei and others added 4 commits December 19, 2023 11:30
When starting a project with `firebase init` with GitHub Workflow,
YAML files were configured with `actions/checkout@v3` previously.

Now, `actions/checkout@v4` has been released,
the template should also be updated to specify the latest version.

https://github.com/actions/checkout/releases/tag/v4.0.0

Co-authored-by: joehan <[email protected]>
* Support export in env parser

* use a noncapture group
@codecov-commenter
Copy link

codecov-commenter commented Dec 26, 2023

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (5f6c816) 54.18% compared to head (bcfa0fb) 54.17%.
Report is 1 commits behind head on master.

Files Patch % Lines
src/frameworks/vite/index.ts 20.00% 4 Missing ⚠️
src/frameworks/sveltekit/index.ts 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6644      +/-   ##
==========================================
- Coverage   54.18%   54.17%   -0.01%     
==========================================
  Files         347      347              
  Lines       24132    24135       +3     
  Branches     4978     4978              
==========================================
  Hits        13075    13075              
- Misses       9856     9859       +3     
  Partials     1201     1201              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@leoortizz leoortizz changed the base branch from next to master January 2, 2024 18:09
@leoortizz leoortizz marked this pull request as ready for review January 3, 2024 14:49
@leoortizz leoortizz self-assigned this Jan 4, 2024
@leoortizz leoortizz enabled auto-merge (squash) January 23, 2024 13:59
@leoortizz leoortizz merged commit d0972f7 into master Jan 23, 2024
@leoortizz leoortizz deleted the leoortizz_sveltekitDevFix branch January 23, 2024 20:13
rrousselGit added a commit that referenced this pull request Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Webframeworks with SvelteKit: Firebase deploy of hosting always uses .env.production Sveltekit dev module variable is true on prod deployment

5 participants