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

Add RBS files for sinatra and sinatra-contrib gems #561

Merged
merged 1 commit into from
May 25, 2024

Conversation

euglena1215
Copy link
Contributor

Generated initial RBS files using the rbs prototype rb command, and added type definitions for both the Sinatra core and contributed extensions.

https://github.com/sinatra/sinatra/tree/v4.0.0

I have set myself as the reviewer for this change because I believe the sinatra gem is an important and widely-used web framework. However, I am not very familiar with sinatra, so if there is someone who uses sinatra regularly and is motivated to maintain these type definitions, I would appreciate it if they could take over the reviewer role.

Generated initial RBS files using the `rbs prototype rb` command,
and added type definitions for both the Sinatra core and contributed extensions.

https://github.com/sinatra/sinatra/tree/v4.0.0

I have set myself as the reviewer for this change because I believe
the sinatra gem is an important and widely-used web framework.
However, I am not very familiar with sinatra, so if there is someone who
uses sinatra regularly and is motivated to maintain these type definitions,
I would appreciate it if they could take over the reviewer role.
Copy link

@euglena1215 Thanks for your contribution!

Please follow the instructions below for each change.
See also: https://github.com/ruby/gem_rbs_collection/blob/main/docs/CONTRIBUTING.md

Available commands

You can use the following commands by commenting on this PR.

  • /merge: Merge this PR if CI passes

sinatra-contrib

This RBS files are newly added.

You can merge this PR immediately if the CI passes.
Just comment /merge to merge this PR.


sinatra

This RBS files are newly added.

You can merge this PR immediately if the CI passes.
Just comment /merge to merge this PR.

@euglena1215
Copy link
Contributor Author

/merge

@github-actions github-actions bot merged commit 4c8290d into ruby:main May 25, 2024
4 checks passed
@euglena1215 euglena1215 deleted the sinatra branch May 25, 2024 16:29
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