Skip to content

Allow self for inline @rbs @ivar: self - #2633

Merged
soutaro merged 1 commit into
ruby:masterfrom
ksss:fix-inline-tAIDENT
Aug 7, 2025
Merged

Allow self for inline @rbs @ivar: self#2633
soutaro merged 1 commit into
ruby:masterfrom
ksss:fix-inline-tAIDENT

Conversation

@ksss

@ksss ksss commented Aug 6, 2025

Copy link
Copy Markdown
Collaborator

Follow-up to #2627

I had set it to prohibit self in instance variables, but this was a mistake.
self should be allowed in instance variables.

@ksss
ksss requested a review from soutaro August 6, 2025 02:22

@soutaro soutaro left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🙏

@soutaro
soutaro added this pull request to the merge queue Aug 7, 2025
Merged via the queue into ruby:master with commit 36847f1 Aug 7, 2025
21 checks passed
@ksss
ksss deleted the fix-inline-tAIDENT branch August 7, 2025 15:18
@ksss ksss added this to the RBS 4.0 milestone Oct 31, 2025
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.

2 participants