Skip to content

Conversation

@snitin315
Copy link
Member

@snitin315 snitin315 commented Jan 21, 2023

  • This is a bugfix
  • This is a feature
  • This is a code refactor
  • This is a test update
  • This is a docs update
  • This is a metadata update

For Bugs and Features; did you add new tests?

Updated existing tests.

Motivation / Use-Case

Resolve TODO.

process.env should be string values.

Screenshot 2023-01-21 at 9 13 06 AM

Breaking Changes

BREAKING_CHANGE: WEBPACK_SERVE value was changed from a boolean true to a string value 'true'.

Additional Info

No.

@snitin315 snitin315 changed the title fix: set WEBPACK_SERVE to string value true fix: set WEBPACK_SERVE to string value 'true' Jan 21, 2023
@codecov
Copy link

codecov bot commented Jan 21, 2023

Codecov Report

Base: 92.14% // Head: 92.14% // No change to project coverage 👍

Coverage data is based on head (2a0d13d) compared to base (ff4c194).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@ Coverage Diff @@ ## next #4718 +/- ## ======================================= Coverage 92.14% 92.14% ======================================= Files 15 15 Lines 1540 1540 Branches 574 574 ======================================= Hits 1419 1419 Misses 112 112 Partials 9 9 
Impacted Files Coverage Δ
lib/Server.js 94.10% <100.00%> (ø)

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 at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@alexander-akait alexander-akait merged commit bafe423 into next Feb 10, 2023
@alexander-akait alexander-akait deleted the update-webpack-serve-env-var branch February 10, 2023 18:37
@alexander-akait
Copy link
Member

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants