Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

lrytz
Copy link
Member

@lrytz lrytz commented Aug 25, 2023

followup to #10439

@lrytz lrytz added this to the 2.13.12 milestone Aug 25, 2023
@lrytz lrytz requested a review from som-snytt August 25, 2023 09:47
@som-snytt
Copy link
Contributor

But wait, there's more!

else
refchecksWarning(member.pos, msg, WarningCategory.OtherNullaryOverride, action)
val cat = if (currentRun.isScala3) WarningCategory.Scala3Migration else WarningCategory.OtherNullaryOverride
refchecksWarning(member.pos, msg, cat, action)
Copy link
Contributor

Choose a reason for hiding this comment

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

this will conflict with my PR for the other ticket, which I can't PR just now because I'm watching Heroes.

@lrytz
Copy link
Member Author

lrytz commented Aug 25, 2023

Rebased on top of #10512

@SethTisue SethTisue added the prio:hi high priority (used only by core team, only near release time) label Aug 25, 2023
@SethTisue SethTisue changed the title Turn more Xsource:3 errors into fatal warnings Turn more -Xsource:3 errors into fatal warnings Aug 26, 2023
Copy link
Member

@SethTisue SethTisue left a comment

Choose a reason for hiding this comment

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

this is primarily a "I'm okay with this going into 2.13.12" review. as in "we don't need to reset the community testing clock". let's still see what Som says

@som-snytt
Copy link
Contributor

I added a commit on my PR to remove the stale check file.

@SethTisue
Copy link
Member

needs rebase

@SethTisue SethTisue merged commit 9bf2653 into scala:2.13.x Aug 30, 2023
@SethTisue SethTisue removed the prio:hi high priority (used only by core team, only near release time) label Aug 30, 2023
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.

3 participants