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

analytics: remove unused analytics. #2752

Merged
merged 1 commit into from Jun 9, 2017

Conversation

Projects
None yet
3 participants
@MikeMcQuaid
Copy link
Member

MikeMcQuaid commented Jun 7, 2017

We didn't end up using the screenview and exception analytics as much as expected so let's remove them and focus on stuff that's formula-specific.

CC @Homebrew/maintainers to check none of you are using these and for thoughts.

analytics: remove unused analytics.
We didn't end up using the `screenview` and `exception` analytics as
much as expected so let's remove them and focus on stuff that's
formula-specific.
@JCount

JCount approved these changes Jun 8, 2017

@MikeMcQuaid MikeMcQuaid merged commit 0e2b043 into Homebrew:master Jun 9, 2017

2 of 3 checks passed

codecov/project 65.15% (-0.01%) compared to e2e578f
Details
codecov/patch Coverage not affected when comparing e2e578f...5367f1b
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@MikeMcQuaid MikeMcQuaid removed the in progress label Jun 9, 2017

@MikeMcQuaid MikeMcQuaid deleted the MikeMcQuaid:remove-unused-analytics branch Jun 9, 2017

@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018

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