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

Release version 3? #338

Closed
tfausak opened this issue Dec 18, 2015 · 23 comments
Closed

Release version 3? #338

tfausak opened this issue Dec 18, 2015 · 23 comments
Assignees
Labels
Milestone

Comments

@tfausak
Copy link
Collaborator

tfausak commented Dec 18, 2015

Since #217 is going into a minor release, the only big-ticket thing associated with the v3.0.0 milestone is #87. #333 fixes that and is ready to go. Is there anything else we want to include with version 3? Nothing else comes to mind for me. What about you, @AaronLasseigne?

@tfausak tfausak added this to the v3.0.0 milestone Dec 18, 2015
@tfausak
Copy link
Collaborator Author

tfausak commented Dec 18, 2015

I should have mentioned:

@tfausak
Copy link
Collaborator Author

tfausak commented Dec 18, 2015

I forgot about #245. I think any fix for that could go with a minor release since it would only add functionality.

@AaronLasseigne
Copy link
Owner

Give me a bit to think about what other stuff might be good to include if anything.

@AaronLasseigne
Copy link
Owner

We might want to consider dropping support for Ruby 1.9.

@tfausak
Copy link
Collaborator Author

tfausak commented Dec 19, 2015

That would have to go along with a major release. Honestly I don't mind continuing to support Ruby 1.9.3. I stand by what I said in #246.

@AaronLasseigne
Copy link
Owner

It's not a big thing but I'm wondering if we should take the opportunity.

@tfausak
Copy link
Collaborator Author

tfausak commented Dec 20, 2015

Is there anything you would like to do that you can't because of Ruby 1.9.3 compatibility? There are a few things that would be nice, but I don't have a strong opinion about them:

  • Starting with Ruby 2 the default encoding is UTF-8. That would allow us to ditch the # coding: utf-8 comments everywhere. However this doesn't really affect us since our source files are all ASCII.
  • Ruby 2 introduced the %i(...) syntax for arrays of symbols. We could use this in literally one non-spec line of code.
  • Ruby 2 introduced keyword arguments. I don't think this would affect us at all.

I feel like we'd be dropping support for no good reason. If we wanted to do something that required Ruby >=2 I could get behind dropping support for 1.9.3.

@tfausak
Copy link
Collaborator Author

tfausak commented Jan 7, 2016

Maybe we should officially drop support for Ruby 1.9 but continue to unofficially support it. In other words, change the documentation and the gem spec, but don't make any backwards-incompatible changes to the code.

@AaronLasseigne
Copy link
Owner

The advantage being that we can then change it later without a major version bump?

@tfausak
Copy link
Collaborator Author

tfausak commented Jan 7, 2016

Yes.

@AaronLasseigne
Copy link
Owner

I'm fine with that approach.

@tfausak
Copy link
Collaborator Author

tfausak commented Jan 8, 2016

We could also take this opportunity to (officially) drop support for ActiveModel 3.2.

@AaronLasseigne
Copy link
Owner

Probably a good idea.

@tfausak
Copy link
Collaborator Author

tfausak commented Jan 8, 2016

Now that #333, #344, and #346 are finished and green, are you ready to merge them and release 3.0?

@tfausak
Copy link
Collaborator Author

tfausak commented Jan 8, 2016

We should add a note to the README about using ActiveInteraction with older versions of Ruby and Rails. Something like:

If you want to use ActiveInteraction with Ruby < 2.0.0 or ActiveModel < 4.0.0, use ActiveInteraction < 3.0.0.

@AaronLasseigne
Copy link
Owner

I think that's reasonable. What section?

@tfausak
Copy link
Collaborator Author

tfausak commented Jan 9, 2016

I was thinking the "Installation" section. That's where we talk about supported versions.

@AaronLasseigne
Copy link
Owner

Works for me.

@AaronLasseigne
Copy link
Owner

I've added it in #348.

@AaronLasseigne
Copy link
Owner

I've started a branch to prep for the 3.0 release. Should we wait until Rails 5 is officially out so we can make sure everything's good?

@tfausak
Copy link
Collaborator Author

tfausak commented Jan 12, 2016

I don't want to. They don't have a release date set and we already work with the beta. I think we'll be fine with respect to Rails 5.

@AaronLasseigne
Copy link
Owner

Works for me. If you're ready we can merge #348, update the release date in the changelog, and I think it's ready to go.

@tfausak tfausak self-assigned this Jan 13, 2016
@tfausak
Copy link
Collaborator Author

tfausak commented Jan 13, 2016

@tfausak tfausak closed this as completed Jan 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants