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

three updates: improved debug messages; pushing status checks; init file quietening #119

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

ghost
Copy link

@ghost ghost commented Jul 20, 2016

No description provided.

@shawn-sterling
Copy link
Owner

Hello, at a glance this looks all good.

Can you clean up the CI Errors?

Thanks

-Shawn

@ghost
Copy link
Author

ghost commented Jul 25, 2016

Hi Shawn,

Yes, sure, ace7a16 committed and passing.

Cheers,
Alex

@ghost
Copy link
Author

ghost commented Jul 27, 2016

Hi Shawn,

Few more fixes/features:

  • added ability to report on thresholds from performance data
  • if Carbon were given zero metrics don't bother connecting
  • bugfix for a log line that was silently throwing errors

...i've cleared up and fixed the lint warnings from flake8. It does however appear that travis' test on 2.6 is failing with a flake8/python3 deprecated warning -- looks like an internal problem at Travis-ci? Could you comment on this?

Ta
Alex

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.

2 participants