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

add badges for github last commit date and commit freq #928

Closed
wants to merge 3 commits into from

Conversation

mskonovalov
Copy link

@mskonovalov mskonovalov commented Apr 1, 2017

Created bages for
Need badges for Github Repo Code frequency or status
#897

  1. latest commit date
    image
    image
    image
    image

  2. commits amount for week/month/year
    image

@paulmelnikow paulmelnikow added the service-badge New or updated service badge label Apr 2, 2017
@paulmelnikow
Copy link
Member

Hi, thanks for your contribution.

The last commit badge is very clear.

I'm not as clear about the frequency badge. When it says "commits | 6k/year", what does that mean?

@mskonovalov
Copy link
Author

It's just amount of commits for last 52 weeks.
Unfortunately API doesn't provide enough features.

@mskonovalov
Copy link
Author

I can add something like "average amount of commits per week for last 52 weeks"

@mskonovalov mskonovalov closed this Apr 2, 2017
@mskonovalov mskonovalov reopened this Apr 2, 2017
@paulmelnikow
Copy link
Member

I'm working on a branch for this. I'll open a new pull request when it's ready.

paulmelnikow added a commit that referenced this pull request Oct 2, 2017
@MarkoPaul0
Copy link

The badge for the last commit is stuck on displaying "last Friday" even though I've made commits every day since. Has anyone run into that issue before? (Note that I set the HTTP cache to 3600 sec to no avail)

@platan
Copy link
Member

platan commented Mar 18, 2018

@MarkoPaul0 Could you provide an example (a repo name)?

@MarkoPaul0
Copy link

MarkoPaul0 commented Mar 18, 2018

@platan, sure thing and thanks for helping out!

@MarkoPaul0
Copy link

Wow I feel really dumb for missing that in the doc. Thanks again @platan!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
service-badge New or updated service badge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants