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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃Ч Remove MultiJSON, JSON and Yajl (Use Oj instead) #3785
馃Ч Remove MultiJSON, JSON and Yajl (Use Oj instead) #3785
Conversation
2ef4b92
to
dea1f84
Compare
dea1f84
to
da177c1
Compare
- Use Oj.load and Oj.dump - Oj.load => rescue StandardError - Oj.dump + RestController => mode: :compat - No 'pretty' option anymore
da177c1
to
0dc68cf
Compare
- Use Oj.load and Oj.dump
ffe4f70
to
b0ca889
Compare
- Use Oj.load and Oj.dump
4dcc9b1
to
7fd9b06
Compare
2b9f750
to
7fe7aff
Compare
CATS have been successfully executed (same configuration as |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall a very nice change which tidies up our codebase.
Sometimes you removed pretty
and added mode: :compat
, sometimes not. Is this on purpose? I added comments to some places where I saw that mode
is not specified. Later I saw this is in many more places, so probably not an issue if it's not needed.
Very cool that you added a Rubocop rule!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will admit I did not look at every individual file. Very excited to remove a dependency from the gemfile!
Oj.load
andOj.dump
Oj.load
=>rescue StandardError
Oj.dump
+RestController
=>mode: :compat
pretty
option anymoreI have reviewed the contributing guide
I have viewed, signed, and submitted the Contributor License Agreement
I have made this pull request to the
main
branchI have run all the unit tests using
bundle exec rake
I have run CF Acceptance Tests