Skip to content
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

Update fiber.rbs #378

Closed
wants to merge 1 commit into from
Closed

Update fiber.rbs #378

wants to merge 1 commit into from

Conversation

sbeckeriv
Copy link
Contributor

Dearest Reviewer,

I was parsing this file and found this typo. I believe these strings need to be capitalized. If I am incorrect please close this pull request.

Once again thank you for your work and time.

Dictated but not reviewed,
Becker

Dearest Reviewer,

I was parsing this file and found this typo. I believe these strings need to be capitalized.  If I am incorrect please close this pull request.

Once again thank you for your work and time.

Dictated but not reviewed,
Becker
@sbeckeriv
Copy link
Contributor Author

After reading this commit I see I am incorrect.

565eb55#diff-688758104ef6487758a411339b13bdbcR27

https://github.com/ruby/rbs/blob/master/docs/syntax.md#type-alias-declaration I did not understand the gravity of this statement. These alias types to be used in the rbs system directly.

I am closing this. I wish it was more obvious that these are type alias like it has to start with ~. ~string. '~int`

@sbeckeriv sbeckeriv closed this Aug 22, 2020
@sbeckeriv sbeckeriv deleted the patch-3 branch August 22, 2020 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant