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

Skip ReactorIntegration on Reactor 3.3.x since it got a built-in one #29

Merged
merged 1 commit into from
May 5, 2019

Conversation

bsideup
Copy link
Contributor

@bsideup bsideup commented May 5, 2019

Since reactor/reactor-core#1682 is merged now, we should not instrument Reactor because it natively integrates with BlockHound via the SPI

@bsideup bsideup added type/enhancement A general enhancement area/spi labels May 5, 2019
@bsideup bsideup added this to the 1.0.0.M3 milestone May 5, 2019
@bsideup bsideup merged commit 12da449 into master May 5, 2019
@bsideup bsideup deleted the detect_builtin_reactor_integration branch May 5, 2019 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/spi type/enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant