Dynflow / dynflow

DYNamic workFLOW orchestration engine
http://dynflow.github.io
MIT License
121 stars 44 forks source link

Use JSON directly in json to msgpack migration #402

Closed adamruzicka closed 2 years ago

adamruzicka commented 2 years ago

MultiJson provides a common interface over several JSON parsers. Sadly some of the parsers have subtle differences.

require 'json'
require 'oj'

example = '{"key": 1637704901.3995261}'
Oj.load(example)['key'].class # => BigDecimal
JSON.parse(example)['key'].class # => Float
adamruzicka commented 2 years ago

Rails 7 dropped support for the classic loader, considering tests are greenish in the zeitwerk PR, I'd say it is related. Let's pin rails below 7 for now and unpin it once the zeitwerk pr gets in