Skip to content

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

Merged
merged 11 commits into from
Jun 11, 2025

Conversation

fkwp
Copy link
Contributor

@fkwp fkwp commented Jun 5, 2025

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:

  • set env variable NODE_ENV=development
  • add --mode development command line parameter to yarn (respectively vite)
  • For the sake of debugging the minify option in vite.config.js is set to false

@fkwp fkwp added the PR-Developer-Experience Release note category. A PR that does not change EC but improves working with the repository. label Jun 5, 2025
@fkwp fkwp added the development build runs yarn build process in development mode label Jun 5, 2025
@fkwp fkwp marked this pull request as ready for review June 5, 2025 22:40
@fkwp fkwp requested a review from a team as a code owner June 5, 2025 22:40
@fkwp fkwp requested a review from toger5 June 5, 2025 22:40
@fkwp fkwp changed the title Add developer build mode to GitHub actions Add developer build mode to GitHub action build-element-call.yaml Jun 6, 2025
@fkwp fkwp removed the development build runs yarn build process in development mode label Jun 6, 2025
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",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"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",

Copy link
Contributor Author

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",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"build:embedded:development": "NODE_ENV=development yarn build:full --config vite-embedded.config.js --mode development",
"build:embedded:development": "yarn build:full --mode development",

Copy link
Contributor Author

@fkwp fkwp Jun 9, 2025

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" ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops, yes

Copy link
Contributor Author

@fkwp fkwp Jun 9, 2025

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 ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

correct

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

addressed by 0bf5c4a

@fkwp fkwp merged commit aa092c2 into livekit Jun 11, 2025
32 checks passed
@toger5
Copy link
Contributor

toger5 commented Jun 12, 2025

Super nice this is merged!!
Will rebase my iOS work onto of this now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development build runs yarn build process in development mode PR-Developer-Experience Release note category. A PR that does not change EC but improves working with the repository.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants