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

Point at library subfolder #1083

Merged
merged 3 commits into from
May 28, 2023
Merged

Conversation

dispalt
Copy link
Contributor

@dispalt dispalt commented May 18, 2023

When everything moved to the subdirectory I think it broke the mapping, I was playing around with something and just noticed it stopped correctly mapping sources.

@nkgm
Copy link

nkgm commented May 22, 2023

+1
Also discussed in #1081

@dispalt
Copy link
Contributor Author

dispalt commented May 25, 2023

Thoughts @japgolly ?

@cquiroz
Copy link
Collaborator

cquiroz commented May 25, 2023

It seems this should be fine. Did you publish locally to test?

@dispalt
Copy link
Contributor Author

dispalt commented May 25, 2023

No, I will do that and report back here

@dispalt
Copy link
Contributor Author

dispalt commented May 27, 2023

Okay tested it end to end, and glad I did because I had the version and prefix flipped. It looks good now though.

@cquiroz
Copy link
Collaborator

cquiroz commented May 27, 2023

Cool, thanks a lot. Could you a add a little not on the changelog?

@dispalt
Copy link
Contributor Author

dispalt commented May 27, 2023

Done

@cquiroz
Copy link
Collaborator

cquiroz commented May 28, 2023

Thanks a lot @dispalt

@cquiroz cquiroz merged commit 2c6bff2 into japgolly:master May 28, 2023
@dispalt dispalt deleted the dan/fix-source-map-uri branch May 28, 2023 04:54
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