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

feat(core): Add inline source-maps #200

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

Conversation

tucker87
Copy link

@tucker87 tucker87 commented Aug 26, 2024

Used babel's inline source mapping tools to carry the source maps all the way through the build chain.

image

Notes:

  1. I could not manage to run the jest tests.
  2. I have only done minor tests so far and would like help finding issues as I'm new to this project.
  3. I'm not sure about the use of eval() but I couldn't find a way around it.
  4. May need to break this out into a new option instead of how I used genSourceMap.
  5. I need to test this with the options API for sure as I'm almost positive that this will be undefined.

Edit: I just saw the Contrubuting.md and will alter my PR as soon as I am free.

@tucker87 tucker87 changed the title Add inline source-maps so user can get error feedback feat(core): Add inline source-maps Aug 31, 2024
@tucker87
Copy link
Author

This is based on PR #196 as I couldn't get it to work without it.

This provides error information in the browser. Including the original source

Removed changes to genSourceMap as I realized it was unrelated

Fixes: FranckFreiburger#198
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.

1 participant