-
Notifications
You must be signed in to change notification settings - Fork 115
Add developer build mode to GitHub action build-element-call.yaml #3323
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
Conversation
package.json
Outdated
@@ -8,7 +8,11 @@ | |||
"dev:embedded": "vite --config vite-embedded.config.js", | |||
"build": "yarn build:full", | |||
"build:full": "NODE_OPTIONS=--max-old-space-size=16384 vite build", | |||
"build:full:production": "yarn build:full", | |||
"build:full:development": "NODE_OPTIONS=--max-old-space-size=16384 NODE_ENV=development vite build --mode development", |
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.
"build:full:development": "NODE_OPTIONS=--max-old-space-size=16384 NODE_ENV=development vite build --mode development", | |
"build:full:development": "yarn build:full --mode development", |
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 by 0bf5c4a
package.json
Outdated
"build:embedded": "yarn build:full --config vite-embedded.config.js", | ||
"build:embedded:production": "yarn build:embedded", | ||
"build:embedded:development": "NODE_ENV=development yarn build:full --config vite-embedded.config.js --mode development", |
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.
"build:embedded:development": "NODE_ENV=development yarn build:full --config vite-embedded.config.js --mode development", | |
"build:embedded:development": "yarn build:full --mode development", |
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.
do you mean "build:embedded:development": "yarn build:embedded --mode development"
?
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.
oops, yes
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.
is NODE_DEV=development
not needed ?
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.
correct
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 by 0bf5c4a
… embedded build into one step
Super nice this is merged!! |
This PR allows to enable the developer build mode for a specific PR by using the label "development build".
Using the developer build mode does:
NODE_ENV=development
--mode development
command line parameter toyarn
(respectivelyvite
)minify
option invite.config.js
is set tofalse