-
Notifications
You must be signed in to change notification settings - Fork 333
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
Wrong number of arguments for dump in 2.7.0 #555
Comments
MihaiOnSoftware
changed the title
Internal wrong number of arguments in 2.7.0
Wrong number of arguments for dump in 2.7.0
Dec 1, 2023
It seems that wrapping the arguments into a hash solves the issue:
If that's intentional it might make sense to include it in the compatibility changes |
Fixed in #553 |
Can that fix be released? |
jcoyne
added a commit
to sul-dlss/sul-bento-app
that referenced
this issue
Dec 4, 2023
jcoyne
added a commit
to sul-dlss/sul-bento-app
that referenced
this issue
Dec 4, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In our code base we have a call to the gem that looks like this:
We end up calling that line indirectly in a spec with data that makes it into:
We just upgraded to 2.7.0 today and it leads to the following error from inside the JSON gem:
Please let me know if you need any more information.
The text was updated successfully, but these errors were encountered: