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 the deprecated call halt_callback_chains_on_return_false from config #18904

Merged
merged 1 commit into from
Jun 21, 2019
Merged

Remove the deprecated call halt_callback_chains_on_return_false from config #18904

merged 1 commit into from
Jun 21, 2019

Conversation

djberg96
Copy link
Contributor

After updating to Rails 5.1.x, I started seeing this deprecation warning:

DEPRECATION WARNING: ActiveSupport.halt_callback_chains_on_return_false= is deprecated and will be removed in Rails 5.2. (called from <top (required)> at /home/dberger/Dev/manageiq-providers-azure-djberg96/spec/manageiq/config/environment.rb:5) ** ManageIQ master, codename: Ivanchuk

As per a conversation with @jrafanie I've removed it.

@bdunne bdunne requested a review from jrafanie June 21, 2019 17:50
@jrafanie
Copy link
Member

jrafanie commented Jun 21, 2019

For reference, it was added temporarily on 5.0 so we could silence an annoying deprecation. Now, it's not needed at all, thanks @djberg96

see: #18456

@jrafanie jrafanie removed their request for review June 21, 2019 17:54
Copy link
Member

@jrafanie jrafanie left a comment

Choose a reason for hiding this comment

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

LGTM

@miq-bot
Copy link
Member

miq-bot commented Jun 21, 2019

Checked commit https://github.com/djberg96/manageiq/commit/04eca4931b75f1b07e146155b951c634e7dab92e with ruby 2.3.3, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍰

@jrafanie jrafanie merged commit 0cca6b6 into ManageIQ:master Jun 21, 2019
@jrafanie jrafanie added this to the Sprint 114 Ending Jun 24, 2019 milestone Jun 21, 2019
@chessbyte chessbyte mentioned this pull request Mar 31, 2020
38 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants