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

Remove sourceMappingURL from dist #42

Open
Triloworld opened this issue Aug 18, 2021 · 3 comments
Open

Remove sourceMappingURL from dist #42

Triloworld opened this issue Aug 18, 2021 · 3 comments

Comments

@Triloworld
Copy link

Getting this error:

DevTools failed to load source map: Could not load content for https://unpkg.com/index.umd.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE

Cause by file in dist folder used in CDN production enviroment:

//# sourceMappingURL=index.umd.js.map

We need min.js version that can be used in production and no additional request is in file

@FRSgit
Copy link
Collaborator

FRSgit commented Sep 7, 2021

Hey!

Are you sure that this is an issue for you? AFAIK requests for source maps happen only if the user has a developer console opened - so in the end that request shouldn't happen for different people than developers (with console opened).

If it's really a problem - there is a possiblity to tweak a build process to support this case. But unfortunately I cannot make it happen in next few weeks, so any PRs would be more than welcome!

@Triloworld
Copy link
Author

Yeah. I read that isn't happen for other people. Just it adds bytes to request even if you don't use it as normal user

@RayBenefield
Copy link

I'd like to second the removal of the sourceMappingURL or at least an alternate version. It bloats the dev console with another warning and if we need source mapping then we can use non-CDN approaches.

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

No branches or pull requests

3 participants