Skip to content

Conversation

@Tethik
Copy link

@Tethik Tethik commented Jan 5, 2018

Fixes #23

@coveralls
Copy link

coveralls commented Jan 5, 2018

Coverage Status

Coverage increased (+1.8%) to 88.235% when pulling 5a7badd on Tethik:master into c2eef75 on Netflix-Skunkworks:master.

@Tethik
Copy link
Author

Tethik commented Jan 5, 2018

Although the functionality is quite confusing and documentation is misleading. The better fix might be to improve the local environment detection. Right now it works on a combination of checking for IS_OFFLINE/IS_LOCAL and the filter_local flag.

For now I updated the documentation.

@coveralls
Copy link

coveralls commented Jan 5, 2018

Coverage Status

Coverage increased (+1.8%) to 88.235% when pulling a621e38 on Tethik:master into c2eef75 on Netflix-Skunkworks:master.

@kevgliss
Copy link
Contributor

lgtm, I agree the documentation is a bit confusing. Thanks for the PR.

@kevgliss kevgliss merged commit 4af6ef9 into Netflix-Skunkworks:master Feb 12, 2018
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.

3 participants