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

Stop generating sourceMap in tsconfig.json #58

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lucasadrianof
Copy link

Stop generating the source map when compiling the project, the src/ folder isn't published to NPM, so the source map was pointing to non-existing files.

These source maps were causing projects using exponential-backoff to break when transpiling the code with webpack.

I created an example showing the error here: https://codesandbox.io/p/sandbox/blissful-orla-9bspqw

image

Stop generating the source map when compiling the project, the src
folder isn't published to NPM, so the source map was pointing to
non-existing files
@lucasadrianof lucasadrianof changed the title fix: stop generating sourceMap in tsconfig.json Stop generating sourceMap in tsconfig.json Apr 26, 2023
@khevamann
Copy link

What is the status on this? This is also throwing tons of warnings for us. Would love to have this merged

@lucasadrianof
Copy link
Author

What is the status on this? This is also throwing tons of warnings for us. Would love to have this merged

To be honest I ended up using a different library on the project I was working on because of this error. However, I decided to keep this PR open just in case it gets merged. I'd be curious to hear about the overall project's status, but I'm not entirely sure if it's actively maintained anymore.

@muratcatal
Copy link

muratcatal commented Sep 4, 2023

Hi @khevamann , would you mind merge and publish a new version for the fix?

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.

3 participants