[ruby-core:60411] [ruby-trunk - Feature #9469] BigDecimal support for yaml serialization

From: rafaelmfranca@...
Date: 2014-02-01 17:17:28 UTC
List: ruby-core #60411
Issue #9469 has been updated by Rafael Fran巽a.


I think we can remove that custom yaml serialization from Rails and we don't need to put in upstream.

Looking in the history of that file I found it was added [6 years ago](https://github.com/rails/rails/commit/191ffc94568ca22885eac7001cae7fef452dcfdd) because at [that date BigDecimal doesn't support `to_yaml`](https://web.archive.org/web/20080828045844/http://dev.rubyonrails.org/ticket/8746).

Since we already support it we can remove without any problem.

----------------------------------------
Feature #9469: BigDecimal support for yaml serialization
https://bugs.ruby-lang.org/issues/9469#change-44880

* Author: Zachary Scott
* Status: Assigned
* Priority: Normal
* Assignee: Kenta Murata
* Category: ext
* Target version: current: 2.2.0
----------------------------------------
Given the following serialization methods in rails/rails: https://github.com/rails/rails/blob/master/activesupport/lib/active_support/core_ext/big_decimal/conversions.rb

I feel like these methods would be better supported in the upstream BigDecimal gem.

How do you feel like adding them @mrkn?



-- 
http://bugs.ruby-lang.org/

In This Thread

Prev Next