The Wayback Machine - https://web.archive.org/web/20220128193139/https://github.com/dotnet/aspnetcore/pull/39842
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

[SignalR] Only minify .min.js files #39842

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[SignalR] Only minify .min.js files #39842

wants to merge 1 commit into from

Conversation

@BrennanConroy
Copy link
Member

@BrennanConroy BrennanConroy commented Jan 28, 2022

Fixes #39829

Our webpack configuration handles calling terser so there is no need for an extra build step that manually calls terser. Additionally, added multiple outputs to the webpack config so we can run minify on only the .min.js files.

We might want to consider backporting this to 6.0 since the non-minified file regressed in 6.0 to ship as minified.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

1 participant