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.

Sign up for GitHub

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

Jump to bottom

Detect `::Rails.application.configure` too #1585

Merged

presidentbeef merged 2 commits into main from colon_colon_rails

Apr 26, 2021

Owner

presidentbeef commented Apr 26, 2021 •

edited

Not just Rails.application.configure.

Probably should normalize constants, at least when they are at the top-level. But that might be too disruptive.

Fixes #1584

presidentbeef and others added 2 commits

April 25, 2021 20:20


          Detect ::Rails.application.configure too

04fd02b

Fixes #1584


          Merge branch 'main' into colon_colon_rails

035a18c

presidentbeef merged commit b470c33 into main

presidentbeef deleted the colon_colon_rails branch

April 26, 2021 04:48

Repository owner locked and limited conversation to collaborators

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.