[dry-configurable] warnings

[jerome]

Question

Hi,
We added the uploadcare-ruby-3.2.0 gem and now we have several warnings each time we started our app.

"vendor/bundle/gems/uploadcare-ruby-3.2.0/lib/uploadcare.rb:29:in <top (required)>' [dry-configurable] default value as positional argument to settings is deprecated and will be removed in the next major version Provide a default:` keyword argument instead"

May be a way to fix it : https://hanamimastery.com/episodes/5-configure-anything-with-dry-configurable
(search keyword "constructor" in the page)

How to fix it ?
Thanks

[dmitrijivanchenko]

Hi, @jidylle!

Thanks for the suggestion!

The warning does not affect functionality, so you can simply disable it by setting parameters in your project:

Dry::Configurable.warn_on_setting_constructor_block false
Dry::Configurable.warn_on_setting_positional_default false

source https://github.com/dry-rb/dry-configurable/blob/master/CHANGELOG.md

Probably we will fix it in next major versions.
Thanks!

[jerome]

Thanks Dmitry for your answer. I tested it, but it did not work. Where do we have to put this 2 lines configuration ? In which file ? Thanks. Le mar. 7 déc. 2021 à 04:54, dmitrijivanchenko ***@***.***> a écrit :

…

Hi, @jidylle <https://github.com/jidylle>! Thanks for the suggestion! The warning does not affect functionality, so you can simply disable it by setting parameters in your project: Dry::Configurable.warn_on_setting_constructor_block falseDry::Configurable.warn_on_setting_positional_default false source https://github.com/dry-rb/dry-configurable/blob/master/CHANGELOG.md Probably we will fix it in next major versions. Thanks! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#103 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAYY4AGZTXTCOSCIYDTO7N3UPXDV3ANCNFSM5JQETDVQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[dmitrijivanchenko]

Hey, @jidylle!

I guess the good place for these lines is a initializer. Have you tried to add them in an existing or new initializer file (in Rails apps it usually located in config/initializers/<your_filename>.rb - smth like config/initializers/dry_configurable.rb)?

Have a nice day!

[jerome]

Thanks Dmitri. This is what I tried (in the initializers folder) but unfortunately the result is the same, the warnings are still displayed :/ (see attached). [image: Capture_d_écran_15_12_2021_17_37.png] Le mar. 14 déc. 2021 à 04:47, dmitrijivanchenko ***@***.***> a écrit :

…

Hey, @jidylle <https://github.com/jidylle>! I guess the good place for these lines is a initializer. Have you tried to add them in an existing or new initializer file (in Rails apps it usually located in config/initializers/<your_filename>.rb - smth like config/initializers/dry_configurable.rb)? Have a nice day! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#103 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAYY4ABS354P5TVPAKX2LWDUQ4ADJANCNFSM5JQETDVQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[jerome]

Hi Dmitry, Another solution to remove this "depecrated" warning? Thank u. Le mer. 15 déc. 2021 à 17:41, Jérôme IDYLLE ***@***.***> a écrit :

…

Thanks Dmitri. This is what I tried (in the initializers folder) but unfortunately the result is the same, the warnings are still displayed :/ (see attached). [image: Capture_d_écran_15_12_2021_17_37.png] Le mar. 14 déc. 2021 à 04:47, dmitrijivanchenko ***@***.***> a écrit : > Hey, @jidylle <https://github.com/jidylle>! > > I guess the good place for these lines is a initializer. Have you tried > to add them in an existing or new initializer file (in Rails apps it > usually located in config/initializers/<your_filename>.rb - smth like > config/initializers/dry_configurable.rb)? > > Have a nice day! > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <#103 (comment)>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AAYY4ABS354P5TVPAKX2LWDUQ4ADJANCNFSM5JQETDVQ> > . > Triage notifications on the go with GitHub Mobile for iOS > <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> > or Android > <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. > >

[dmitrijivanchenko]

Hi, @jidylle !

Sorry for late reply! Yes, you are right that initializer won't work because they are loaded after gems (uploadcare gem).
Another hack you can do is placing these two strings in the config/application.rb file.

But you should place them along with require 'dry-configurable' before all gems load string.
See the pic

I also believe that you can move these lines (8-11) to a separate file and require this new file in the config/application.rb for more readability and decreasing size of the config/application.rb file.

Have a nice day!

[jerome]

It works. Thank you Dmitri. Le mer. 29 déc. 2021 à 00:59, dmitrijivanchenko ***@***.***> a écrit :

…

Hi, @jidylle <https://github.com/jidylle> ! Sorry for late reply! Yes, you are right that initializer won't work because they are loaded after gems (uploadcare gem). Another hack you can do is placing these two strings in the config/application.rb file. But you should place them along with require 'dry-configurable' before all gems load string. See the pic [image: dry-conf-hack] <https://user-images.githubusercontent.com/29576452/147628675-12c26f38-61ec-4c36-b7d6-afbe78115325.png> I also believe that you can move these lines (8-11) to a separate file and require this new file in the config/application.rb for more readability and decreasing size of the config/application.rb file. Have a nice day! — Reply to this email directly, view it on GitHub <#103 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAYY4ADV2EJFTWXS7W4G5CTUTKISRANCNFSM5JQETDVQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>