Skip to content

Conversation

@tk0miya
Copy link
Contributor

@tk0miya tk0miya commented Sep 11, 2024

No description provided.

#
class CSV::Table[out Elem] < Object
include Enumerable[untyped]
include Enumerable[Elem]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is this change correct? If my understanding is correct, Elem means a row of the table. So #each and Enumerable should use Elem, right?

Copy link
Member

@soutaro soutaro left a comment

Choose a reason for hiding this comment

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

OK. Give it a try! 👍

@soutaro soutaro added this pull request to the merge queue Sep 13, 2024
@soutaro soutaro added this to the RBS 3.6 milestone Sep 13, 2024
github-merge-queue bot pushed a commit that referenced this pull request Sep 13, 2024
stdlib: Add types for CSV::Table#each
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 13, 2024
@soutaro soutaro added this pull request to the merge queue Sep 13, 2024
Merged via the queue into ruby:master with commit 684b730 Sep 13, 2024
@tk0miya tk0miya deleted the csv_table_each branch September 13, 2024 13:43
@soutaro soutaro added the Released PRs already included in the released version label Sep 17, 2024
ksss added a commit to ksss/gem_rbs_collection that referenced this pull request Feb 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Released PRs already included in the released version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants