ruby / gem_rbs_collection

A collection of RBS for gems.
MIT License
255 stars 106 forks source link

add ericgpks as a reviewer to some rails dependency gems #611

Closed ericgpks closed 3 months ago

ericgpks commented 3 months ago

By reading this issue https://github.com/ruby/gem_rbs_collection/issues/558, I would like to join as a reviewer in some gems. I add me as a reviewer in this PR.

github-actions[bot] commented 3 months ago

@ericgpks 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.


aws-sdk-s3

You changed RBS files for an existing gem. You need to get approval from the reviewers of this gem.

@ksss, please review this pull request. If this change is acceptable, please make a review comment including APPROVE from here. Screen Shot 2024-03-19 at 14 13 36

After that, the PR author or the reviewers can merge this PR. Just comment /merge to merge this PR.


slack-notifier

You changed RBS files for an existing gem. This gem does not have reviewers. So you can merge this PR immediately if the CI passes. We recommend you add yourself to the reviewers for this gem.

ksss commented 3 months ago

@ericgpks Thank you for expressing your intention. However, I am sorry, but I am considering removing the aws-sdk gem from gem_rbs_collection. (I haven't deleted it yet because I haven't been able to verify what will happen if it's removed.) Even if you are added as a reviewer, your first task will likely be to review the removal.

ref: https://github.com/ruby/gem_rbs_collection/issues/511

ericgpks commented 3 months ago

@ksss Thank you for sharing the situation. I got it.

github-actions[bot] commented 3 months ago

Thanks for your review, @ksss!

@ericgpks, @ksss This PR is ready to be merged. Just comment /merge to merge this PR.

ericgpks commented 3 months ago

/merge