Skip to content
This repository has been archived by the owner on Nov 8, 2020. It is now read-only.

Issue creation as a contribution #1

Closed
jeaye opened this issue Aug 9, 2018 · 6 comments
Closed

Issue creation as a contribution #1

jeaye opened this issue Aug 9, 2018 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@jeaye
Copy link

jeaye commented Aug 9, 2018

The Github profile page also takes into account issue creation as part of the contributions, but it seems github-contribs does not. Is there any reason why this wasn't included? Can it be added?

@lourot
Copy link
Member

lourot commented Aug 9, 2018

Hi @jeaye :)

Is there any reason why this wasn't included?

No special reason, I just didn't need it for https://github.com/AurelienLourot/ghuser.io and I wanted to avoid unnecessary HTTP GETs because of the rate limit.

Can it be added?

Yes :) I'll try to do it today. I'll probably make it a --flag and not change the default behavior.

@lourot lourot self-assigned this Aug 9, 2018
@lourot lourot added the enhancement New feature or request label Aug 9, 2018
@jeaye
Copy link
Author

jeaye commented Aug 9, 2018

Sweet! Thanks for the quick and helpful reply. An optional flag for this sounds good to me. :)

@lourot
Copy link
Member

lourot commented Aug 9, 2018

Bear with me please :) I have implemented the whole thing but at the last moment I discovered that it wasn't very reliable. So I dug a bit and this is what happens: bd745c8#diff-04c6e90faac2675aa89e2176d2eec7d8R72

TL;DR "hot issues" need to be fetched differently from "non-hot" ones.

I have now also written a failing unit test for this and I need to implement this part as well. I hope I won't hit anything else.

lourot added a commit that referenced this issue Aug 9, 2018
@lourot
Copy link
Member

lourot commented Aug 9, 2018

done :)

$ sudo npm install -g @ghuser/github-contribs@2.2.0
$ github-contribs --verbose --issues --since 2018-08-08 jeaye
✔ Fetched first day at GitHub: 2011-09-17.
⚠ Be patient. The whole process might take up to a few seconds...
2018-08-09: (hot issues) zulip/docker-zulip
2018-08-08: (hot issues) AurelienLourot/github-contribs
✔ Fetched all commits, PRs and issues.
2 repo(s) found:
zulip/docker-zulip
AurelienLourot/github-contribs

@lourot lourot assigned jeaye and unassigned lourot Aug 9, 2018
@jeaye
Copy link
Author

jeaye commented Aug 9, 2018

Awesome job! This worked very well for me. Thank you! :)

@jeaye jeaye closed this as completed Aug 9, 2018
@lourot
Copy link
Member

lourot commented Aug 9, 2018

Thank you :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants