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

Rollback the rollback to Scala.js 1.5.1 #4048

Merged
merged 1 commit into from
Nov 19, 2021
Merged

Rollback the rollback to Scala.js 1.5.1 #4048

merged 1 commit into from
Nov 19, 2021

Conversation

rossabaker
Copy link
Member

The build on #4018 was broken and merged anyway. Scala.js compatibility has been good enough, and enough of the ecosystem has moved on, I no longer think it's worth trying to keep this one pinned.

This reverts commit fae2bb8, reversing
changes made to 0b01db8.
@satorg
Copy link
Contributor

satorg commented Nov 19, 2021

Yes, I can confirm: I tried running sbt ++2.12.15 clean validateAllJS locally before and after merging #4018.
It succeeds right before but fails on the merge commit.

Copy link
Contributor

@satorg satorg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GitHub actions have succeeded – seems it fixes the build indeed.

@rossabaker
Copy link
Member Author

Since the build is broken and affecting a couple other PRs, I'm going to merge this hastily. If anyone doesn't like it, speak up here before the next release. 😄

@rossabaker rossabaker merged commit 63a4454 into main Nov 19, 2021
@rossabaker rossabaker deleted the revert-4018 branch November 19, 2021 04:17
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