config icon indicating copy to clipboard operation
config copied to clipboard

Rename Settings to Config and display deprecation warning until next major version bump

Open pkuczynski opened this issue 10 years ago • 4 comments

As we renamed the gem from RailsConfig to Config I think we should also rename the global variable name from Settings to Config, keeping both access points available and displaying deprecation warning until the next major version bump.

@fredwu what do you think?

pkuczynski avatar Aug 17 '15 11:08 pkuczynski

This sounds good to me. But could it potentially crash with a rails class? I'm not sure if there is already a Config class or not... If not, it would be beneficial to the consistency for the user.

vinceve avatar Oct 03 '15 12:10 vinceve

Hi there,

I just had a look at this library and as a heavy Sinatra user the first thing I thought was that Settings will inevitably cause problems with Sinatra's own configuration accessor, settings. If I'm going to use this I'd run a fork based on that alone. Imagine differentiating variables based on case! (o_º)

Regards, iain

yb66 avatar Jan 16 '19 00:01 yb66

I think we already have Sinatra as part of our test scope and it seems to work fine with current setup...

pkuczynski avatar Jan 16 '19 10:01 pkuczynski

@pkuczynski Fair enough, I'm just saying that it looks like the kind of thing that if another dev on a team came to me with two vars only differing by case then I'd be worried that a bug will appear. Seems like an accident waiting to happen.

Regards, iain

yb66 avatar Jan 16 '19 10:01 yb66