Fix deprecation warning from Rails 7.2, use kwarg for bold option in log subscriber

elastic / elasticsearch-rails

Elasticsearch integrations for ActiveModel/Record and Ruby on Rails

Apache License 2.0

3.08k stars 798 forks source link

Fix deprecation warning from Rails 7.2, use kwarg for bold option in log subscriber #1058

Closed matthutchinson closed 1 year ago

matthutchinson commented 1 year ago

Why?

Fixes this warning

DEPRECATION WARNING: Bolding log text with a positional boolean is deprecated and will be removed in Rails 7.2. Use an option hash instead (eg. color("my text", :red, bold: true))

This stems from the use of the color function in log_subscriber.rb

cla-checker-service[bot] commented 1 year ago

❌ Author of the following commits did not sign a Contributor Agreement: 028cad99c9cf3a47df83f8f5a69c6c3b2126721f

Please, read and sign the above mentioned agreement if you want to contribute to this project