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 Data Collector #48

Merged
merged 1 commit into from
Dec 8, 2017
Merged

Fix Data Collector #48

merged 1 commit into from
Dec 8, 2017

Conversation

ChristianRiesen
Copy link
Contributor

@Oliboy50 That last minute change slipped a mistake into the code. This fixes it. Please kill the 3.0.1 tag when you merge this. Thanks and sorry for the screw up.

Copy link

@pabedu pabedu left a comment

Choose a reason for hiding this comment

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

😱 😄

Copy link
Contributor

@b-viguier b-viguier left a comment

Choose a reason for hiding this comment

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

Integration wasn't so helpful in this case 😭

@ChristianRiesen
Copy link
Contributor Author

Good thing I immediately updated to the public version, which caused my code to scream at me in an instant :)

@b-viguier b-viguier merged commit a25f2ca into BedrockStreaming:master Dec 8, 2017
@Oliboy50
Copy link
Member

Oliboy50 commented Dec 8, 2017

to https://github.com/M6Web/AmqpBundle/tree/v3.0.2
(we didn't want to destroy a tag, even if it was broken... because of packagist/satis stuff)

@ChristianRiesen
Copy link
Contributor Author

Thank you and sorry again, that should not have slipped past me.

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.

5 participants