Warn when you make missing attribute searchable

[ellnix]

Pull Request

Related issue

Fixes #168

What does this PR do?

• In addition to adding a warning as suggested in Error or warning when trying to use undefined attributes #168 it also adds a #logger method to MeiliSearch::Rails which is similar to Rails having its own Rails#logger

PR checklist

Please check if your PR fulfills the following requirements:

• Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
• Have you read the contributing guidelines?
• Have you made sure that the title is accurate and descriptive of the changes?

[brunoocasali]

Thanks a lot @ellnix, for all of your work here!

I made some comments I would like you to reevaluate before we can merge this.

[ellnix]

I believe I fixed the issues suggested, I had made some really strange decisions on this PR.

I also removed one place where I used the new ruby 3 method definition syntax which would have broken compatibility with ruby 2.x in the test suite.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ef5d96c) 88.58% compared to head (007d085) 88.73%.

❗ Current head 007d085 differs from pull request most recent head 645cf76. Consider uploading reports for the commit 645cf76 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #293      +/-   ##
==========================================
+ Coverage   88.58%   88.73%   +0.15%     
==========================================
  Files          10       10              
  Lines         648      657       +9     
==========================================
+ Hits          574      583       +9     
  Misses         74       74              

Files	Coverage Δ
lib/meilisearch-rails.rb	90.53% <100.00%> (+0.17%)	⬆️
lib/meilisearch/rails/pagination.rb	93.75% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[brunoocasali]

I believe I fixed the issues suggested, I had made some really strange decisions on this PR.

I also removed one place where I used the new ruby 3 method definition syntax which would have broken compatibility with ruby 2.x in the test suite.

Nice, as soon you fix them we will be good to go, thanks a lot for your effort @ellnix!

[ellnix]

@brunoocasali this should be good to go.

[brunoocasali]

@ellnix thanks for this fix 🎉

Thanks a lot for your patience and your smart work here ❤️

[ellnix]

bors merge

[meili-bors]

Build succeeded:

• integration-tests (Rails 6.1 with Ruby 2.7)
• integration-tests (Rails 6.1 with Ruby 3.0)
• integration-tests (Rails 6.1 with Ruby 3.1)
• integration-tests (Rails 7 with Ruby 2.7)
• integration-tests (Rails 7 with Ruby 3.0)
• integration-tests (Rails 7 with Ruby 3.1)
• linter-check
• smoke-test