Skip to content
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

WIP: Re-write #54

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

WIP: Re-write #54

wants to merge 20 commits into from

Conversation

Rucknar
Copy link
Contributor

@Rucknar Rucknar commented Sep 23, 2020

A number of issues with the prior code of this repository meant that it was difficult to maintain and some metrics couldn't be catered for. As a result, we've taken the time to re-factor this and have the google/go-github sdk do the leg work for us.
The result is a significantly simpler package and with some new metrics thrown in for good measure.

  • The number of additions is so high because of vendoring, in reality the codebase is much smaller now.

@Rucknar
Copy link
Contributor Author

Rucknar commented Sep 23, 2020

Outstanding

  • Add pagination support back in
  • Add metrics for number of commits
  • Add metrics for github actions
  • Add metrics for Release Downloads (in prior version)
  • Failure mode, capture rate limit info should errors be encountered

@Rucknar
Copy link
Contributor Author

Rucknar commented Sep 28, 2020

  • Add pagination support back in
  • Add metrics for number of commits
  • Add metrics for github actions
  • Add metrics for Release Downloads (in prior version)
  • Failure mode, capture rate limit info should errors be encountered
  • Add pagination support to enriched metric collection
  • Fix bug in release count metrics
  • Add secondary release metrics (Downloads per release)
  • Increase test coverage
  • Re-instate full build testing
  • Add in go routines to enable paralell processing of metrics.
  • Compare output from existing version for same test organisations

@Rucknar
Copy link
Contributor Author

Rucknar commented Sep 29, 2020

Resolves #52, Resolves #51, Resolves #37, Resolves #30, Resolves #22, Resolves #25, Resolves #12, Resolves #1

@kvz
Copy link

kvz commented Jan 23, 2021

This is looking very good! Can't wait to try it out :)

I'd like to monitor which collaborator has how many issues assigned & closed, will that be possible with the rewrite? Tried to check the diff but GitHub is cowardly refusing to render the "Jump to" table because of the vendoring.

In my mind it would be something like:

github_issues{user="kvz",repo="transloadit/uppy",state="open"} 4
github_issues{user="kvz",repo="transloadit/uppy",state="closed"} 1

Not sure if this a possible/great way of doing it--just a first idea.

@pinpox
Copy link

pinpox commented Jan 21, 2022

Is this still in progress? I'd like to get the date of the last commit of specific branches, is that already supported or will be anytime soon?

@gamunu
Copy link

gamunu commented Jan 23, 2023

Is this abandoned? we are interested to see these features merged to master especially GitHub actions metrics

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.

None yet

4 participants