Skip to content

[browser][CoreCLR] JS minification in Release#126877

Merged
pavelsavara merged 1 commit intodotnet:mainfrom
pavelsavara:fix_minification
Apr 14, 2026
Merged

[browser][CoreCLR] JS minification in Release#126877
pavelsavara merged 1 commit intodotnet:mainfrom
pavelsavara:fix_minification

Conversation

@pavelsavara
Copy link
Copy Markdown
Member

No description provided.

@pavelsavara pavelsavara added this to the 11.0.0 milestone Apr 14, 2026
@pavelsavara pavelsavara self-assigned this Apr 14, 2026
Copilot AI review requested due to automatic review settings April 14, 2026 14:10
@pavelsavara pavelsavara added arch-wasm WebAssembly architecture area-System.Runtime.InteropServices.JavaScript os-browser Browser variant of arch-wasm labels Apr 14, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates build-time configuration detection so Release builds are identified case-insensitively (supporting JS minification behavior in Release).

Changes:

  • Make isDebug default to "Debug" when Configuration is unset
  • Compare Configuration case-insensitively against "release"

Comment thread src/native/rollup.config.defines.js
@pavelsavara pavelsavara marked this pull request as ready for review April 14, 2026 16:22
@pavelsavara
Copy link
Copy Markdown
Member Author

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

Labels

arch-wasm WebAssembly architecture area-System.Runtime.InteropServices.JavaScript os-browser Browser variant of arch-wasm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants