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

[GitHub] Repository size badge #1174

Merged
merged 5 commits into from
Oct 14, 2017
Merged

Conversation

PyvesB
Copy link
Member

@PyvesB PyvesB commented Oct 14, 2017

Hello,

As discussed in the review comments of #1160, here is a pull request for a new repository size badge.

repo size

Cheers,

Pyves

@paulmelnikow paulmelnikow added the service-badge Accepted and actionable changes, features, and bugs label Oct 14, 2017
@paulmelnikow
Copy link
Member

Thanks! Could you take a look at the CI failure? There are two jobs. The first job should pass, though the second will fail because of #979.

@PyvesB
Copy link
Member Author

PyvesB commented Oct 14, 2017

Done, it was simply an ESLint related issue, which I had not run locally!

@paulmelnikow
Copy link
Member

After applying the fix from #1118 locally, I'm seeing a handful of failures when I run the Github service tests. We should figure out why these are unreliable at some point! Anyway, the failures are not related to this change.

@paulmelnikow paulmelnikow merged commit b0f2eed into badges:master Oct 14, 2017
@PyvesB PyvesB deleted the pull_request branch October 15, 2017 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
service-badge Accepted and actionable changes, features, and bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants