Skip to content
This repository was archived by the owner on Sep 17, 2021. It is now read-only.

Conversation

@redixin
Copy link
Contributor

@redixin redixin commented Mar 17, 2017

There little typo in docs and signature. Method report_auditor_changes receive auditor instance, not items list.

@redixin
Copy link
Contributor Author

redixin commented Mar 17, 2017

@kalpatel01 please take a look ^

@coveralls
Copy link

Coverage Status

Coverage remained the same at 56.62% when pulling 487b0a7 on redixin:custom_alerters_fix_docs into ea1c45e on Netflix:develop.

@scriptsrc
Copy link
Contributor

I haven't heard from @kalpatel01.

I'm going to merge this in.

@scriptsrc scriptsrc merged commit 1b26995 into Netflix:develop Mar 23, 2017
@redixin redixin deleted the custom_alerters_fix_docs branch March 23, 2017 08:55
This was referenced Apr 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants