Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
pkuczynski opened this issue Aug 17, 2015 · 4 comments

Comments

@pkuczynski
Copy link
Member

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 pkuczynski changed the title Renaming Settings to Config and display deprecation warning until next major version bump Rename Settings to Config and display deprecation warning until next major version bump Aug 17, 2015
@vinceve
Copy link

vinceve commented Oct 3, 2015

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.

@yb66
Copy link

yb66 commented Jan 16, 2019

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

@pkuczynski
Copy link
Member Author

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

@yb66
Copy link

yb66 commented Jan 16, 2019

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants