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

Fix `remove_index` to be able to remove expression indexes #26655

Merged
merged 1 commit into from Feb 13, 2017

Conversation

Projects
None yet
5 participants
@kamipo
Member

kamipo commented Sep 28, 2016

Fixes #26635, #26641.

@rails-bot

This comment has been minimized.

rails-bot commented Sep 28, 2016

r? @schneems

(@rails-bot has picked a reviewer for you, use r? to override)

@kamipo kamipo force-pushed the kamipo:fix_remove_expression_index branch 2 times, most recently Sep 29, 2016

@kamipo kamipo force-pushed the kamipo:fix_remove_expression_index branch 2 times, most recently Oct 28, 2016

@kamipo kamipo force-pushed the kamipo:fix_remove_expression_index branch to 4e35d9f Dec 24, 2016

@rafaelfranca rafaelfranca merged commit 1c968b4 into rails:master Feb 13, 2017

2 checks passed

codeclimate no new or fixed issues
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@kamipo kamipo deleted the kamipo:fix_remove_expression_index branch Feb 13, 2017

rafaelfranca added a commit that referenced this pull request Feb 13, 2017

Merge pull request #26655 from kamipo/fix_remove_expression_index
Fix `remove_index` to be able to remove expression indexes
@rafaelfranca

This comment has been minimized.

Member

rafaelfranca commented Feb 13, 2017

Backported in 646dfab

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment