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

Remove support for Rails 3/Rails 3.1 apps. #157

Merged
merged 1 commit into from Sep 1, 2014
Merged

Conversation

lucasmazza
Copy link
Contributor

This is a follow up of the latest discussion at #124 and I'm opening this so it won't be left for dead in a branch.

The idea is to drop support for Rails 3 and ship both jQuery 1.11.x and 2.1.x inside the gem. I believe we can ship both JavaScript files inside vendor/assets/javascripts and everyone will be able to //= require either jquery2 or jquery on their apps without any hassle.

I'll also gonna need to update the rake task that download the latest jQuery version to support both 1.11.x and 2.1.x releases, since there isn't a specific URL to grab the latest jQuery 2 script like we do with http://code.jquery.com/jquery.js and friends. The expected workflow would be to update the version.rb first and then running the task to download all the assets for the specific versions.

/summon @JangoSteve

@@ -1,3 +1,7 @@
## Master

- Removed support for Rails 3 and Rails 3.1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably read "Rails <= 4.2" (or something to that effect) to be less confusing to the user reading this 😄

@chancancode
Copy link
Member

🤘

rafaelfranca added a commit that referenced this pull request Sep 1, 2014
Remove support for Rails 3/Rails 3.1 apps.
@rafaelfranca rafaelfranca merged commit 35d96fb into master Sep 1, 2014
@rafaelfranca rafaelfranca deleted the rails-4-only branch September 1, 2014 17:08
@lucasmazza lucasmazza mentioned this pull request Sep 1, 2014
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants