-
Notifications
You must be signed in to change notification settings - Fork 584
feature: switch no longer being removed #23288
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
Conversation
It no longer produces experimental warnings. Fixes Perl#23282
Is it also no longer deprecated? |
Indeed: the warning seems to have been removed in 2dd81b7. The |
It can always be re-deprecated if removal is again attempted, but at the very least, the documentation should indicate that it is discouraged and will receive no further development. |
pod/perldeprecation.pod implies to me that if we deprecate something, we are going to remove it. I doubt(ed) we'll be removing these features, so I removed the deprecations. And it was applied as is. It might be worth reinstating the deprecation message and re-wording it to indicate discouragement instead of deprecation. There's still some references to these being experimental, I'm not sure if those need re-wording, or we considered it the remnants of a failed experiment (the apparatus exploded, leaving a mess all over the lab.) |
I'd like to suggest a kind of hypothetical __ |
We discussed the topic during the PSC meeting today, and concluded:
|
Also, I confused the |
The information is already available in Perl::Version::Bumper. It might be interesting to come up with a proper interface to access that data (it's only used internally by the module so far.) |
There is still a reference to the smartmatch operator being removed in perlglossary; to switch being removed in perlfaq7; and to smartmatch and switch being experimental in perlsyn and perlop which I noted in #23286 (comment). These may need a more comprehensive update to indicate the current state and plans for these two features. (perlglossary and perlfaq7 are maintained on CPAN so will need to be updated there) |
98455cd
to
63db6c7
Compare
I think I covered the suggested changes. |
It no longer produces experimental warnings.
Fixes #23282