Skip to content

Commit

Permalink
use 4-0-stable arel branch to get bundle working again.
Browse files Browse the repository at this point in the history
this resolves the following error when running `bundle install`

```
ArgumentError: wrong number of arguments (2 for 1)
    /Users/senny/Projects/rails/.bundle/bundler/gems/arel-454a25f18c95/lib/arel/crud.rb:5:in `compile_update'
```

The build is still broken with lots of:

```
ArgumentError: wrong number of arguments (2 for 1)
    /Users/senny/Projects/rails/.bundle/bundler/gems/arel-454a25f18c95/lib/arel/crud.rb:5:in `compile_update'
```
  • Loading branch information
senny committed Nov 19, 2013
1 parent 2b18e22 commit 3053fee
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Gemfile
Expand Up @@ -12,7 +12,7 @@ gem 'bcrypt-ruby', '~> 3.1.2'
gem 'jquery-rails', '~> 2.2.0' gem 'jquery-rails', '~> 2.2.0'
gem 'turbolinks' gem 'turbolinks'
gem 'coffee-rails', '~> 4.0.0' gem 'coffee-rails', '~> 4.0.0'
gem 'arel', github: 'rails/arel', branch: 'master' gem 'arel', github: 'rails/arel', branch: '4-0-stable'


# This needs to be with require false to avoid # This needs to be with require false to avoid
# it being automatically loaded by sprockets # it being automatically loaded by sprockets
Expand Down

5 comments on commit 3053fee

@senny
Copy link
Member Author

@senny senny commented on 3053fee Nov 19, 2013

Choose a reason for hiding this comment

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

@rafaelfranca
Copy link
Member

Choose a reason for hiding this comment

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

Master should use master. If it is broken with arel master we need to fix

@senny
Copy link
Member Author

@senny senny commented on 3053fee Nov 19, 2013

Choose a reason for hiding this comment

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

@rafaelfranca should I update the activerecord gem to depend on arel 5.0.0?

@senny
Copy link
Member Author

@senny senny commented on 3053fee Nov 19, 2013

Choose a reason for hiding this comment

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

Updated with a6f58a5

@rafaelfranca
Copy link
Member

Choose a reason for hiding this comment

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

👍

Please sign in to comment.