Skip to content
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

Don't produce compressed assets at build time and instead do it as part of the nginx build #2995

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

hughns
Copy link
Member

@hughns hughns commented Feb 6, 2025

The core vite build shouldn't be responsible for implementing optimisations that are specific to a particular distribution mechanism. e.g. static compressed files for nginx.

This way the core build is "clean".

Instead, the packager/distributor of the build should be responsible for it. e.g. the nginx docker distribution should do any optimisations that it chooses.

Part of #2994

…rt of the nginx build

The core vite build shouldn't be responsible for implementing optimisations that are specific to a particular distribution mechanism. e.g. static compressed files for nginx.

This way the core build is "clean".

Instead, the packager/distributor of the build should be responsible for it. e.g. the nginx docker distribution should do any optimisations that it chooses.
@hughns hughns requested a review from a team as a code owner February 6, 2025 10:52
@hughns hughns requested a review from toger5 February 6, 2025 10:52
@hughns hughns added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Feb 6, 2025
@hughns hughns enabled auto-merge (squash) February 6, 2025 12:43
Copy link
Contributor

@fkwp fkwp left a comment

Choose a reason for hiding this comment

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

lgtm

@hughns hughns merged commit 0128f2f into livekit Feb 6, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants