-
Notifications
You must be signed in to change notification settings - Fork 20
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
Version 5.2.1 shows jazz_fingers
as application name
#25
Comments
Which version of Rails are you using? |
Sorry, forgot to mention in the issue. I'm using Rails 6.0.3.2. |
Ah, I can see it now. 57c97fa tries to fix a deprecation warning, but removed the Should we return a default like this instead of maybe throwing an exception or displaying a warning? In which cases would this not return a value? |
Ouch. Good catch. Could you open a PR? |
I guess just adding return would be enough. If we can get the application name let use it otherwise use the default which is the gem name. |
Version 5.2.0:
Version 5.2.1:
The text was updated successfully, but these errors were encountered: