Skip to content

Rename features all and any with a keyword_ prefix #23105

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

Merged
merged 2 commits into from
Mar 14, 2025

Conversation

leonerd
Copy link
Contributor

@leonerd leonerd commented Mar 14, 2025

(Fixes #23104)

This avoids confusion between use feature 'all' and the footgun of use feature ':all'.


  • This set of changes requires a perldelta entry, and it is included

@leonerd
Copy link
Contributor Author

leonerd commented Mar 14, 2025

@book As original requester, any review comments? I've added a perldelta now too

@leonerd leonerd added the squash-before-merge Author must squash the commits down before merging to blead label Mar 14, 2025
@book
Copy link
Contributor

book commented Mar 14, 2025

@book As original requester, any review comments? I've added a perldelta now too

Looks good to me.

leonerd added 2 commits March 14, 2025 14:31
…word_any' and 'keyword_all'

This avoids confusion with the `use feature ':all'` ability

(As requested by Perl#23104)
@leonerd leonerd force-pushed the rename-keywords-any-all branch from e2a19b7 to 28be74a Compare March 14, 2025 14:32
@leonerd leonerd removed the squash-before-merge Author must squash the commits down before merging to blead label Mar 14, 2025
@leonerd leonerd merged commit bf81de7 into Perl:blead Mar 14, 2025
33 checks passed
@leonerd leonerd deleted the rename-keywords-any-all branch March 14, 2025 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename the "all" and "any" features to "keyword_all" and "keyword_any"
3 participants