Skip to content
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

Revert removing BigDecimal.new in version 1.4 for very old version of Rails #114

Closed
mrkn opened this issue Dec 14, 2018 · 10 comments
Closed
Labels

Comments

@mrkn
Copy link
Member

mrkn commented Dec 14, 2018

I decided to revert removing BigDecimal.new on bigdecimal 1.4 because very old version of Rails depends on it.

After releasing Ruby 2.6, I will release bigdecimal 1.5 without BigDecimal.new.

Reference

@mrkn mrkn added the ruby-2.6 label Dec 14, 2018
@mrkn mrkn closed this as completed in f138dfa Dec 14, 2018
@ioquatix
Copy link
Member

Good idea. A lot of code depends on this. In theory, it shouldn't be removed until Ruby 3.0 because it's a backwards incompatible change?

@mrkn
Copy link
Member Author

mrkn commented Dec 21, 2018

@ioquatix

it shouldn't be removed until Ruby 3.0 because it's a backwards incompatible change?

No, I don't think so because bigdecimal is just a standard library, but not ruby's core.

I have a plan to release version 1.5.0 with removing BigDecimal.new.
I'll import the latest version of bigdecimal in Ruby 2.7.

Users can control what version of bigdecimal is used by calling gem method or writing the version contraint in Gemfile.

@ioquatix
Copy link
Member

If you are using semantic versioning you should not remove public interface in minor version bump. It should be 2.0.0 release to change or remove existing interface in backwards incompatible way.

@mrkn
Copy link
Member Author

mrkn commented Dec 21, 2018

bigdecimal doesn’t employ semantic versioning.

@ioquatix
Copy link
Member

What kind of versioning does it use?

@JonRowe
Copy link

JonRowe commented Dec 21, 2018

An awful lot of people will use ~> versioning as semantic versioning is the norm in Ruby. I urge you to reconsider making a breaking change like this in a minor release, after all whats a version number? Releasing a 2.0.0 for a change like this in a core library has little in the way of downside and enormous upsides for the community.

@ioquatix
Copy link
Member

All I can say, is that I agree that:

  • We should be using semantic versioning.
  • Removing a public method in a minor version bump is unexpected and broke stuff.
  • Some level of compatibility is required going forward.

@mrkn
Copy link
Member Author

mrkn commented Dec 23, 2018

I think it is good idea that version 2.0.0 follows version 1.4.0 because I can release bigdecimal 3.0.0 when CRuby 3.0 is released.

So, these are my decision:

  • bigdecimal doesn't employ semantic versioning.
  • The version after 1.4.0 is 2.0.0, and it will be released soon after CRuby version 2.6.
  • The second digit of the version number is increased every year according to CRuby's version (e.g. bigdecimal 2.1.0 will be released with CRuby 2.7)
  • The version 3.0.0 will be released with CRuby version 3.0.

@ruby ruby locked as resolved and limited conversation to collaborators Dec 23, 2018
@ruby ruby unlocked this conversation Dec 23, 2018
@mrkn
Copy link
Member Author

mrkn commented Dec 23, 2018

This issue was closed and isn't about versioning topic.
I don't want to continue to talk about versioning here, so I lock this issue.

@ruby ruby locked as resolved and limited conversation to collaborators Dec 23, 2018
@ioquatix
Copy link
Member

I've created https://bugs.ruby-lang.org/issues/15456 for further discussion. I invite everyone to get involved.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

3 participants