search

ruby / gem_rbs_collection

A collection of RBS for gems.
MIT License
246 stars 101 forks source link

add: RBS signeture of ActiveRecord::Encryption::EncryptableRecord module #544

Closed rhiroe closed 2 months ago

rhiroe commented 2 months ago

https://github.com/rails/rails/blob/v7.0.8.1/activerecord/lib/active_record/encryption/encryptable_record.rb

Generated automatically by typeprof.

The method definitions in the class_methods block of ActiveSupport::Concern have been rewritten to the ClassMethods module.

github-actions[bot] commented 2 months ago

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

activerecord

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.

github-actions[bot] commented 2 months ago

Thanks for your review, @rhiroe!

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

github-actions[bot] commented 2 months ago

Thanks for your review, @soutaro!

@rhiroe, @soutaro This PR is ready to be merged. Just comment /merge to merge this PR.

github-actions[bot] commented 2 months ago

Thanks for your review, @ksss!

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

rhiroe commented 2 months ago

@soutaro @ksss Thanks for the review! I have completed the corrections.

github-actions[bot] commented 2 months ago

Thanks for your review, @tk0miya!

@rhiroe, @tk0miya This PR is ready to be merged. Just comment /merge to merge this PR.

github-actions[bot] commented 2 months ago

Thanks for your review, @rhiroe!

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

github-actions[bot] commented 2 months ago

Thanks for your review, @tk0miya!

@rhiroe, @tk0miya This PR is ready to be merged. Just comment /merge to merge this PR.

rhiroe commented 2 months ago

I will wait for a additional reviews until about 17:00 JST, and if nothing happens, I will run /merge.

rhiroe commented 2 months ago

/merge