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

Fix typo in lib/flipper-active_record.rb #437

Merged

Conversation

bendb-instacart
Copy link
Contributor

Per @mike101 in #432:

I'm not sure if it's just me, but this change seems to have broken Rails for me using Rails 6 and the latest version of this gem with this PR included. I get an error now where bundler is trying to load the gem, and I get an error that it can't find activesupport/lazy_load_hooks.

I typo'd my PR, and forgot the underscore in active_record. I don't know how the tests passed, but they did. I have manually verified this fix, and would encourage reviewers to do the same.

@jnunemaker jnunemaker merged commit e2b1f5d into flippercloud:master Sep 13, 2019
@bendb-instacart bendb-instacart deleted the fix_flipper_activerecord branch September 13, 2019 23:33
@jnunemaker
Copy link
Collaborator

0.17.1 is out with this fix.

bendb-instacart added a commit to bendb-instacart/flipper that referenced this pull request Sep 14, 2019
This will guard against regressions like that fixed in flippercloud#437.
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.

2 participants