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

Allow for debug and production builds of Monaco JS files #17

Merged
merged 2 commits into from
Mar 23, 2022

Conversation

wkordalski
Copy link
Contributor

In Rust debug mode, debug build is used, thus minification and name mangling step is skipped.
In Rust release mode, JS files are minified allowing for maximum performance.

I'm not sure if this is the best approach. Maybe we should use some feature flag instead of basing on build mode?
Nevertheless we should allow for using builds with nonminified files and without names mangled, because it really helps debugging and hacking Monaco editor.

In Rust debug mode, debug build is used, thus minification
and obfuscation step is skipped.
In Rust release mode, JS files are minified allowing for
maximum performance.
Copy link
Owner

@siku2 siku2 left a comment

Choose a reason for hiding this comment

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

I like this approach. I think we should use js/debug and js/prod subfolders though rather than multiple js-xyz directories.

@wkordalski
Copy link
Contributor Author

Done. And changed prod to release to be more consistent with Rust naming.

@siku2
Copy link
Owner

siku2 commented Mar 23, 2022

Done. And changed prod to release to be more consistent with Rust naming.

Great, I was gonna suggest that too but didn't want to be too pedantic :)

@siku2 siku2 merged commit d9d1a1b into siku2:master Mar 23, 2022
@wkordalski wkordalski deleted the debug-and-release-builds branch March 23, 2022 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants