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

If `add_filter()` is already available, use it in `WP_CLI::add_hook()` #3384

Merged
merged 1 commit into from Sep 8, 2016

Conversation

1 participant
@danielbachhuber
Copy link
Member

commented Sep 8, 2016

This gracefully lets WP-CLI use the tentative WP_Hook implementation
in WP 4.7. Even if WP_Hook gets pulled, it's better to use
add_filter() if it's available.

Fixes #3382

If `add_filter()` is already available, use it in `WP_CLI::add_hook()`
This gracefully lets WP-CLI use the tentative `WP_Hook` implementation
in WP 4.7. Even if `WP_Hook` gets pulled, it's better to use
`add_filter()` if it's available.

@danielbachhuber danielbachhuber added this to the 0.25.0 milestone Sep 8, 2016

@danielbachhuber danielbachhuber merged commit ca22fd1 into master Sep 8, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@danielbachhuber danielbachhuber deleted the 3382-add-action-early branch Sep 8, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.