-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Handle experimental features in releases #11774
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
Comments
I am strictly against a flag. The whole point of language imports is that you don't need a flag a anymore! Flags are obscure, we never know which ones are set, and we don't know how to set them in many situations. For instance, quickly: how do you set a flag in a worksheet? in a notebook? in sbt? in mill? Language imports much better: Clear, readable, self-documenting, easy to discover. What we should do is enable the experimental language import only for unstable releases. |
We should at least have a flag to enable or disable non-release/release on a non-release artifact to be able to test it properly. @sjrd proposed to do it this way. I had originally proposed the opposite, a blag that can be used in nightlies to make it run as if it were a release artifact. This is important to test if the experimental features are properly disabled in the release build before getting we release. |
I think having a |
Maybe it would be more clear if it's called something more like -Ystable-features-only or -Yno-unstable-features |
The standard library depends on the experimental feature |
Hence #11845 |
Fix #11774: only enable experimental features for snapshot and nightly
Re-open because of the revert #11916. |
I can take a look unless someone else wants to do it. In which case please re-assign. |
To properly handle the usage restrictions on experimental features we need:
-Xenable-experimental
). This flag will enable the use of experimental features trough language imports or language flags.-Xenable-experimental
-Xenable-experimental
The text was updated successfully, but these errors were encountered: