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

Fix Scala.JS source map URIs #14243

Merged
merged 1 commit into from Jan 10, 2022
Merged

Fix Scala.JS source map URIs #14243

merged 1 commit into from Jan 10, 2022

Conversation

hugo-vrijswijk
Copy link
Contributor

@hugo-vrijswijk hugo-vrijswijk commented Jan 10, 2022

Dotty publishes release versions without a v in the tag, but the Scala.JS source map URI configuration is set with a v prefix. This causes incorrect source map URLs like: https://raw.githubusercontent.com/lampepfl/dotty/v3.1.0/library/src/scala/IArray.scala (this is in the current latest release JAR).

This commit fixes the source map URIs to no longer use the v prefix. Resulting in a URL like: https://raw.githubusercontent.com/lampepfl/dotty/3.1.0/library/src/scala/IArray.scala (for a next release)

Dotty publishes release versions without a `v` in the tag, but the Scala.JS source map URI configuration is published with a `v` prefix. This causes incorrect source map URLs like: https://raw.githubusercontent.com/lampepfl/dotty/v3.1.0/library/src/scala/IArray.scala (this is in the current releases JAR).

This commit fixes the source map URIs to no longer use the `v` prefix.
@smarter
Copy link
Member

smarter commented Jan 10, 2022

Thanks for spotting this!

@smarter smarter merged commit 1e7ad75 into scala:master Jan 10, 2022
@hugo-vrijswijk hugo-vrijswijk deleted the fix/scalajs-source-uri branch January 10, 2022 18:13
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.

None yet

2 participants