Skip to content

Fix Scala.js source map URI to point to GitHub; drop Scala 2.11#97

Merged
rossabaker merged 2 commits intoLog4s:masterfrom
hugo-vrijswijk:scalajs-sourcemapUri
Mar 13, 2026
Merged

Fix Scala.js source map URI to point to GitHub; drop Scala 2.11#97
rossabaker merged 2 commits intoLog4s:masterfrom
hugo-vrijswijk:scalajs-sourcemapUri

Conversation

@hugo-vrijswijk
Copy link
Contributor

Adds the TypelevelScalaJSGitHubPlugin to the build, which configures the Scala.js source map URI to point to GitHub. Sets the -scalajs-mapSourceURI: scalacOption so source-maps will point to GitHub, rather than a local file which will not exist for end users of log4s.

sbt:Log4s Root> show coreJS/scalacOptions
---
[info] * -P:scalajs:mapSourceURI:file:/Users/hvanrijswijk1/ws/log4s/->https://raw.githubusercontent.com/Log4s/log4s/1f23b6c7cbc7400cce1f6670422461b62fa0211e/

@rossabaker
Copy link
Contributor

There's going to be some toil here. Scala 2.11 hasn't been touched in 7 years, so I'd be fine dropping that in a minor version.

Adds the `TypelevelScalaJSGitHubPlugin` to the build, which configures the Scala.js source map URI to point to GitHub. Sets the `-scalajs-mapSourceURI:` scalacOption so source-maps will point to GitHub, rather than a local file which will not exist for end users of log4s.

```sbt
sbt:Log4s Root> show coreJS/scalacOptions
---
[info] * -P:scalajs:mapSourceURI:file:/Users/hvanrijswijk1/ws/log4s/->https://raw.githubusercontent.com/Log4s/log4s/1f23b6c7cbc7400cce1f6670422461b62fa0211e/
```
@hugo-vrijswijk
Copy link
Contributor Author

There's going to be some toil here. Scala 2.11 hasn't been touched in 7 years, so I'd be fine dropping that in a minor version.

Fine by me, I can remove it in the same PR and update to other more recent Scala versions at the same time

@rossabaker rossabaker changed the title Fix Scala.js source map URI to point to GitHub Fix Scala.js source map URI to point to GitHub; drop Scala 2.11 Mar 13, 2026
@rossabaker rossabaker merged commit 4e0fceb into Log4s:master Mar 13, 2026
9 checks passed
@rossabaker
Copy link
Contributor

I released 1.11.0-RC1 with this. If this scratched your itch, we can do a proper release next week.

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.

2 participants