-
Notifications
You must be signed in to change notification settings - Fork 1k
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 migration guide #2015
Update migration guide #2015
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Wait, the tests fail because a link is broken:
|
8f6a001
to
501709e
Compare
@b-studios in case this is ready, I put a link to it into the 2.13.6 release notes :) |
@b-studios I think it's acceptable to merge and remove the fence in a post-merge commit. It's unlikely there's an intention behind it, since it's missing in the original migration guide repository, so I see no reason to fret over it too much if there's a hurry for release notes inclusion... |
501709e
to
ab894b9
Compare
ab894b9
to
88e7780
Compare
Should be ready to merge now. |
Thanks @adpi2 ! |
Fix more links
Port latest changes:
-no-indent
instead of-noindent
scalacenter/scala-3-migration-guide#204 by @unkarjedy