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.

Sign up for GitHub

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

Jump to bottom

Logging plugin infrastructure #34

Merged
merged 54 commits into from Feb 16, 2018
Merged

Logging plugin infrastructure #34

merged 54 commits into from Feb 16, 2018

Conversation

danpalmer
Copy link
Contributor

Would appreciate a review of the approach.

Still todo:

  • Make the logging API more useful for different kinds of logging (i.e. Prometheus metrics).
  • Log useful stuff from around the code...
    • Status codes from webhooks.
    • Status codes from feeds.

@coveralls
Copy link

coveralls commented Feb 14, 2018 •
edited

Coverage Status

Coverage increased (+0.8%) to 99.149% when pulling 64c4017 on logging-plugins into 566e702 on master.

@danpalmer danpalmer merged commit 4a528f3 into master Feb 16, 2018
@danpalmer danpalmer deleted the logging-plugins branch February 16, 2018 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@prophile prophile Awaiting requested review from prophile

Assignees
No one assigned
Labels
For Review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@danpalmer @coveralls