Anonymous Analytics Collection #374

Merged
merged 17 commits into from Sep 14, 2016

Conversation

Projects
None yet
2 participants
@ryuichis
Contributor

ryuichis commented Sep 13, 2016

OCLint is provided free of charge and run entirely by contributors. As a result, we do not have the resources to do detailed studies of how OCLint is deployed and applied in the real-world.

This pull request gives us the possibility of collecting anonymous analytics about how OCLint is used. This is completely anonymous and is only used for helping us improve the product. The data we collect (to mention a few, the operating system and its version, which languages OCLint analyzes on, OCLint configurations, rule configurations, etc) will help us decide the features and environments that we keep or deprecate support for, and help us prioritize current work. To see exactly how and what we collect, check out the Analytics.cpp file in this pull request.

We value privacy and we treat privacy very seriously. The data we collect is governed by my consulting company's Privacy Policy. For the data we collect in OCLint, it's totally anonymous, and we don't collect any personally identifiable information. Again, check out the changes in this pull request for details.

In addition, helping us improve our product is voluntary, and leaving it on is appreciated. However, it can be opted out by giving -no-analytics argument to OCLint:

oclint -no-analytics [other options] <source0> [... <sourceN>]

Alternatively, if OCLint is built from source code locally, this will prevent analytics module from ever being built:

./build -no-analytics
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Sep 13, 2016

Coverage Status

Coverage decreased (-0.2%) to 82.722% when pulling 9c79920 on ryuichis:analytics into 9cbfd87 on oclint:master.

Coverage Status

Coverage decreased (-0.2%) to 82.722% when pulling 9c79920 on ryuichis:analytics into 9cbfd87 on oclint:master.

@ryuichis ryuichis merged commit 5889b5e into oclint:master Sep 14, 2016

1 of 2 checks passed

coverage/coveralls Coverage decreased (-0.2%) to 82.722%
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@ryuichis ryuichis deleted the ryuichis:analytics branch Sep 14, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment