-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Include source map into theia image by default #13729
Comments
I think there are some specific reasons why we remove source maps from che-theia image, @benoitf what do you think? |
well, first I do not think it's kind of a blocker. second, yes it's to have a minimal image size. |
It blocks me at the moment, because I'm trying to remove the code, which removes sourcemaps from image and can't get success with it at the moment. It would be good to have the environment variable which will include sourcemaps during theia image build (to include sourcemaps and cdn support or not), enabled by default. |
IMHO you're blocked is different that there is blocker in product. |
Issues go stale after Mark the issue as fresh with If this issue is safe to close now please do so. Moderators: Add |
+1 |
Issues go stale after Mark the issue as fresh with If this issue is safe to close now please do so. Moderators: Add |
Description
There is a need to debug a client side (theia in workspace, not a dedicate hosted instance) in theia, but when image is build, all files needed for developmen are removed due to production cleaning step. This blocks debugging client side and instigates editing theia image and implies an extra work to build image, push it into own registry and edit deployment configuration in minikube cluster that might resets after cluster stops.
The text was updated successfully, but these errors were encountered: