Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Deprecate aws-sdk v1 #1997

Closed
maclover7 opened this issue Sep 9, 2015 · 8 comments
Closed

Deprecate aws-sdk v1 #1997

maclover7 opened this issue Sep 9, 2015 · 8 comments

Comments

@maclover7
Copy link
Contributor

As said by @tute, we need to deprecate the use of aws-sdk v1, as major version 2 has been released (currently at v2.1.19).

Steps we need to follow for this:

  • Release a minor version which includes a deprecation warning if the user is still on aws-sdk v1
  • Release a major version which includes the actual deprecation of aws-sdk v1 (this is a breaking change)
@tute
Copy link
Contributor

tute commented Sep 10, 2015

For context, I'd deprecate it if Amazon is deprecating it. Is it? If so, when? If not, it's an arguable step to take.

@jyurek
Copy link

jyurek commented Sep 18, 2015

Agreed. I'd only want to deprecate it if Amazon is deprecating it.

@maclover7
Copy link
Contributor Author

aws/aws-sdk-ruby#924

@tute
Copy link
Contributor

tute commented Sep 18, 2015

Thanks! It sounds like Rails 5 will come before they come up with a decision anyway.

@maclover7
Copy link
Contributor Author

yep, so let's have Rails 5 + dropping Rails 3.2 + any other backwards incompatible changes as apart of Paperclip 5.0.0. Should we release a minor version of Paperclip that includes deprecation warnings?

@tute
Copy link
Contributor

tute commented Sep 18, 2015

People on Rails 2.3.x may use paperclip 2.7, people on Rails 3 may use paperclip 3, and so on. I don't think we need deprecation warnings for Rails.

If there are other more subtle deprecations or changes of functionality that could be surprising, I'm all for warnings.

@tute
Copy link
Contributor

tute commented Nov 12, 2015

I'll bite the bullet and call for a removal of AWS V1 for paperclip 5. trevorrowe says in aws/aws-sdk-ruby#924 (comment):

we took a lot of effort to ensure that version 1 and version 2 could be used in the same project allowing users to upgrade at gradually and at will

v1 users can continue to use paperclip 4.x.

@maclover7
Copy link
Contributor Author

Closing in favor of #2049.

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

No branches or pull requests

3 participants