-
Notifications
You must be signed in to change notification settings - Fork 1.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
Make @experimental final #20260
Make @experimental final #20260
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.
We should rebase #20222 on this branch to ensure this change fixes the issue.
Otherwise LGTM
8d41005
to
1d7f304
Compare
@noti0na1 Could you squash and rebase? |
The current failure is fixed in #20264 |
1d7f304
to
fc43000
Compare
@nicolasstucki #20222 has been rebased on to this PR and passes all the tests. |
Closed, the change will be put together with #20222. |
I extend /** Used to cross-compile with Scala 2. */
class experimental extends scala.annotation.experimental /** Used to cross-compile with Scala 3. */
class experimental extends scala.annotation.StaticAnnotation |
Use a type alias instead. |
As discussed in #20223, this PR makes @experimental final.
Should unblock #20222.