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

Update correct line numbers in LocationSpec #1152

Closed
wants to merge 1 commit into from

Conversation

som-snytt
Copy link
Contributor

The test breaks with scala/scala3#17428

I'm not sure what the protocol is for a project participating in the community build, where this behavior change was noticed.

Parking this here for reference.

@etorreborre
Copy link
Owner

Thanks @som-snytt. I adjusted in the fix in 947e823 so that the spec passes both specs2 current CI and the community build for the next scala version.

@etorreborre etorreborre closed this May 7, 2023
@etorreborre
Copy link
Owner

etorreborre commented May 7, 2023

One thing I don't know though is how the community build selects a proper specs2 version for its build.

@som-snytt
Copy link
Contributor Author

Clever! Thanks, and of course I have no idea if/when the PR might be accepted on the Scala 3 side.

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