Skip to content

Conversation

noti0na1
Copy link
Member

As discussed in #20223, this PR makes @experimental final.

Should unblock #20222.

@noti0na1 noti0na1 added area:library Standard library area:experimental needs-minor-release This PR cannot be merged until the next minor release labels Apr 24, 2024
@noti0na1 noti0na1 requested a review from nicolasstucki April 24, 2024 18:20
Copy link
Contributor

@nicolasstucki nicolasstucki left a 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

@nicolasstucki
Copy link
Contributor

@noti0na1 Could you squash and rebase?

@nicolasstucki
Copy link
Contributor

The current failure is fixed in #20264

@noti0na1 noti0na1 force-pushed the make-experimental-final branch from 1d7f304 to fc43000 Compare April 25, 2024 14:43
@noti0na1
Copy link
Member Author

@nicolasstucki #20222 has been rebased on to this PR and passes all the tests.

@nicolasstucki nicolasstucki self-requested a review April 26, 2024 18:29
@noti0na1
Copy link
Member Author

Closed, the change will be put together with #20222.

@noti0na1 noti0na1 closed this Apr 30, 2024
@joroKr21
Copy link
Member

joroKr21 commented May 5, 2024

I extend experimental to cross-compile with Scala 2 (due to experimental APIs in quotes.reflect)

/** 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

@sjrd
Copy link
Member

sjrd commented May 5, 2024

Use a type alias instead.

@sjrd sjrd deleted the make-experimental-final branch May 5, 2024 04:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:experimental area:library Standard library needs-minor-release This PR cannot be merged until the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants