Skip to content

Remove an unnecessary each call#72

Closed
nobu wants to merge 1 commit into
ruby:masterfrom
nobu:scan
Closed

Remove an unnecessary each call#72
nobu wants to merge 1 commit into
ruby:masterfrom
nobu:scan

Conversation

@nobu

@nobu nobu commented Jun 26, 2025

Copy link
Copy Markdown
Member

String#split yields each element if the block is given, since ruby 2.6.

@olleolleolle olleolleolle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! EDIT: Ah. The CI matrix does have a Ruby 2.5 in it.

@ioquatix

ioquatix commented Dec 9, 2025

Copy link
Copy Markdown
Member

Will we drop 2.5 support?

@hsbt

hsbt commented Jan 9, 2026

Copy link
Copy Markdown
Member

There is no reason to change this just to use Ruby 2.6 or later. Dropping support for older Rubies should only happen when there are features or bug fixes that are only available in newer versions.

@hsbt hsbt closed this Jan 9, 2026
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.

4 participants