-
Notifications
You must be signed in to change notification settings - Fork 120
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
Make fluentd a suggest requirement rather than mandatory #229
Comments
What's the reasoning behind making this change? @rokob @coryvirok what do you guys think? |
@cordoval's suggestion sounds like the right thing to do to me. Most people using rollbar-php aren't going to use the fluent connector, so there's no reason it should be a required dependency. |
The way this works also is i use kubernetes and I have to have the freedom to just channel the exceptions however we wanted. Fluentd is not bad, just not a must have. |
Ready for review in PR #230 |
closing this as #230 is LGTM too 👍 |
No description provided.
The text was updated successfully, but these errors were encountered: