-
Notifications
You must be signed in to change notification settings - Fork 0
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
Activity analytics #2
Comments
Well, I don't like this overall
And maybe we will want to introduce separate URLs for every user and repo later (if we'll add more time-expensive metrics and/or if the existing metrics suddenly become expensive). Also I think there was a problem with |
Yes, using the array was a bad idea; dictionary is better. Your proposed format seems to be missing the point of the feature, though — we want something like "average number of commits over some periods of time", and it's nowhere to be found in your dicts.
It also occurred to me that we should specify how exactly the commits are counted. With DVCS it's possible for a developer to code away for a week without pushing anything, and then suddenly we have all those commits pushed to all those repositories. How should we handle this? I see two approaches:
Even though, I'd vote for the second approach as more "fair". |
I just haven't shown some keys from the dictionaries. I've updated my examples to show them. Regarding the commit counting - I'd say we don't care at this moment. We can use only data provided by Github API and if it provides both kinds of dates - that's fine; if not - that's fine too. |
I've replaced the format in the draft with the one you suggested. Thanks! I also added a sentence clarifying how we count commits. |
Closing the issue due to the stabilization of version 1 of the API. |
This is feature two of the three that @ForNeVeR originally envisioned.
Gestalt should monitor commits to the repositories belonging to the organization and report per-user and per-repo statistics (probably with weekly and monthly averages).
The text was updated successfully, but these errors were encountered: