Skip to content

replace NODE_ENV during Vite build#6644

Merged
leoortizz merged 10 commits intomasterfrom
leoortizz_sveltekitDevFix
Jan 23, 2024
Merged

replace NODE_ENV during Vite build#6644
leoortizz merged 10 commits intomasterfrom
leoortizz_sveltekitDevFix

Conversation

@leoortizz
Copy link
Copy Markdown
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 <joehanley@google.com>
* Support export in env parser

* use a noncapture group
@codecov-commenter
Copy link
Copy Markdown

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