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
Remove analytics from Bower, fixes #1102 #2150
Conversation
Code change: LGTM 👍 broader question: If someone runs bower i... with silent... and they have not configured their prefs yet (I am thinking of the run on CI test case) can we just assume no tracking? The only reason I ask is because losing these metrics is a fairly big hit imho. |
Ship Ahoy! All looking good 🚢 |
@samccone We already assumed no analytics on CI. Plus I don't know any way we used these metrics to develop bower.. And nowadays we also "freeze" bower releases with bundledDependencies. Developers reporting issues on Github do far better job than analytics of how many times someone wrote |
sounds good to me @sheerun 🚢 away |
👍 Thanks for the work here |
👍 |
@mojoaxel Good catch :) |
They caused more issues than useful they were. Instead, we'll focus on fetching statistics from NPM registry to watch Bower's popularity, and GitHub stars over time to watch packages' popularity.
7d875fd
to
3e3b642
Compare
Remove analytics from Bower, fixes #1102
They caused more issues than useful they were.
Instead, we'll focus on fetching statistics from NPM registry to watch Bower's popularity, and GitHub stars over time to watch packages' popularity.
http://bower.io/stats/ is already updated to use only non-google-analytics data.