-
Notifications
You must be signed in to change notification settings - Fork 12k
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
Is there a way to specify which source maps to create with production build? #2533
Comments
I think #2519 fixes this. |
@filipesilva I see this is waiting for a sync ("This branch is out-of-date with the base branch"). Any ETA on that? |
@filipesilva Just to clarify, are TS sourcemaps for |
@filipesilva thanks!! |
@tomwanzek always. That was the original intention but we forgot to correctly configure |
@filipesilva And if we don't want map files in prod is it possible to not have them? |
@maxailloud not something that's specifically planned atm. Although to be honest there have been a fair number of these requests to further customise minor aspects of builds, so probably in the future we'll just have way more flags there. For now though, I wouldn't expect it soon as there are way bigger fish to fry. In your specific case I'd make the file copy script ignore/delete the map files when deploying, there should be no problem. |
@filipesilva Yeah sure I can definitley understand that there is more important stuff to do. Thank you for your answer. |
I also need an option to build without source maps, seems basic |
Fixed by #3113 |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Since there is no access to the webpack.config.js (and it's devtool setting) for angular-cli, is there any way to control which type of sourcemaps are generated in the production build? I want the production build and compression, etc but I want usable TS sources as well so I can troubleshoot the code once it is deployed. This that possible?
The text was updated successfully, but these errors were encountered: