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

chore: Update scala native to 0.5.1 #761

Merged
merged 2 commits into from
Apr 18, 2024
Merged

Conversation

tgodzik
Copy link
Contributor

@tgodzik tgodzik commented Apr 17, 2024

Fixes #760

@tgodzik
Copy link
Contributor Author

tgodzik commented Apr 17, 2024

This is blocked by the issue in LTS

@mzuehlke
Copy link
Contributor

Just thinking loud, would we loose anything if we move munit to 3.3.3 and uncomment the failing test ?
The people who are on Scala LTS and use munit would still not get the right error message.
We just have to remember to enable the tests once the fix landed in LTS

@mpilquist
Copy link

At minimum, munit needs to move to 3.3.x to consume the scalacheck artifacts.

@tgodzik
Copy link
Contributor Author

tgodzik commented Apr 17, 2024

Maybe yeah. I will do that in this pr

|""".stripMargin
)
}
if (BuildInfo.scalaVersion != "3.3.3") {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ignored the tests specifically for 3.3.3 so this should not be ignored for the next LTS

Copy link
Contributor

Choose a reason for hiding this comment

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

looks good 👍

another possibility would be to have in each of the test as first line

assume(BuildInfo.scalaVersion != "3.3.3")

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's a nicer solution, thanks!

@tgodzik
Copy link
Contributor Author

tgodzik commented Apr 18, 2024

I will go ahead and merge this, then release next and hopefully last milestone

@tgodzik tgodzik merged commit 967d354 into scalameta:main Apr 18, 2024
9 checks passed
@tgodzik tgodzik deleted the update-0.5 branch April 18, 2024 09:10
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.

Publish for Scala Native 0.5, please
3 participants