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

shout out blog post #88

Closed
gr2m opened this issue Apr 18, 2016 · 9 comments · Fixed by hoodiehq/hood.ie#276
Closed

shout out blog post #88

gr2m opened this issue Apr 18, 2016 · 9 comments · Fixed by hoodiehq/hood.ie#276
Assignees

Comments

@gr2m
Copy link
Member

gr2m commented Apr 18, 2016

hey @sjnorth, thanks to @finnp’s work, we now have a script that gets more data for us to give shout outs. Here’s a summary for contirbutions since last shout out, can you work with that? It says who contributed for the first time, who reviewed, and you can give some extra shout outs based on contributions, like Finn’s work on the internal architecture migration and Cecily on the TGIFs. Let me know if you have any farther question

Charlotteis worked on 7 repositories.

HipsterBrown worked on 8 repositories.

  • All-time total: 26 merged PRs.
  • merged 6 pull requests
  • reviewed 19 pull requests
  • commented 22 times

HopeMobile worked on hoodiehq/hoodie-client

KaLaUnlimited worked on hoodiehq/hoodie-account-server

NeekyRabbit worked on hoodiehq/editorial

SonjaHeinen worked on hoodiehq/hood.ie

  • All-time total: 0 merged PRs.
  • commented 3 times

alisabzevari worked on hoodiehq/hoodie-account-server

antoinelyset worked on hoodiehq/hoodie-connection-status

artivilla worked on hoodiehq/hoodie-account-server

boennemann worked on hoodiehq/hoodie-account-server

bryanbierce worked on hoodiehq/hoodie-account-server

  • All-time total: 0 merged PRs.
  • commented 3 times

caityc worked on hoodiehq/hoodie-account

chrifpa worked on hoodiehq/hoodie-account-client

chrisdemars worked on hoodiehq/hood.ie

  • All-time total: 0 merged PRs.
  • commented 5 times

christophwitzko worked on hoodiehq/hoodie-account-client

coderbyheart worked on hoodiehq/pouchdb-hoodie-sync

codyzu worked on hoodiehq/hoodie-account-server

coveralls worked on hoodiehq/hoodie-connection-status

dar5hak worked on hoodiehq/hoodie-server

doublerebel worked on hoodiehq/hoodie-server

espy worked on hoodiehq/editorial

  • All-time total: 20 merged PRs.
  • commented 2 times

finnp worked on 11 repositories.

gitsemere worked on hoodiehq/hoodie-client-task

gr2m worked on 21 repositories.

greenkeeperio-bot worked on 9 repositories.

hintsa worked on 3 repositories.

  • All-time total: 1 merged PRs.
  • First PR on the hoodie project!
  • landed effect less chane
  • commented 3 times

hyperperforator worked on hoodiehq/hoodie-account-server

hzoo worked on hoodiehq/hoodie-account-server

inator worked on hoodiehq/hoodie

janl worked on 3 repositories.

  • All-time total: 58 merged PRs.
  • merged 1 pull requests
  • reviewed 1 pull requests
  • commented 14 times

jkaplon worked on hoodiehq/faq

jo-porter worked on hoodiehq/editorial

jsimplicio worked on hoodiehq/editorial

  • All-time total: 7 merged PRs.
  • commented 2 times

jywarren worked on hoodiehq/hoodie-app-tracker

  • All-time total: 0 merged PRs.
  • commented 2 times

kazekagegaara worked on hoodiehq/pouchdb-admins

kmcrayton7 worked on hoodiehq/editorial

  • All-time total: 0 merged PRs.
  • commented 2 times

konson worked on hoodiehq/hoodie-store-server

  • All-time total: 1 merged PRs.
  • First PR on the hoodie project!
  • landed fix git clone URL
  • commented 2 times

kprenesti worked on hoodiehq/documentation and hoodiehq/documentation

marisalaneous worked on 3 repositories.

matsieftw worked on hoodiehq/editorial

  • All-time total: 0 merged PRs.
  • commented 2 times

nickcolley worked on 5 repositories.

  • All-time total: 15 merged PRs.
  • merged 2 pull requests
  • reviewed 3 pull requests
  • commented 8 times

nolanlawson worked on hoodiehq/hoodie-account-server

pawan92 worked on hoodiehq/hood.ie and hoodiehq/hood.ie

raquelxmoss worked on hoodiehq/documentation and hoodiehq/documentation

renrutnnej worked on 3 repositories.

roubaeli worked on hoodiehq/hoodie-account-server

rubymorillo worked on hoodiehq/hood.ie and hoodiehq/hood.ie

sebbel worked on 5 repositories.

sergi worked on hoodiehq/hoodie-server

  • All-time total: 0 merged PRs.
  • commented 2 times

shyNotSly worked on hoodiehq/hoodie-account

sjnorth worked on hoodiehq/editorial

skeskali worked on hoodiehq/hood.ie

stevemasta34 worked on hoodiehq/hoodie-account-server

  • All-time total: 1 merged PRs.
  • commented 2 times

summerspirit worked on hoodiehq/editorial

  • All-time total: 0 merged PRs.
  • commented 2 times

texassiga worked on hoodiehq/hoodie-account-server

  • All-time total: 0 merged PRs.
  • commented 2 times

timblack1 worked on hoodiehq/hoodie-server

turcapat worked on hoodiehq/editorial

  • All-time total: 0 merged PRs.
  • commented 10 times

willholley worked on hoodiehq/faq

@gr2m
Copy link
Member Author

gr2m commented Apr 18, 2016

@finnp should it say that @codyzu had his first contirbution to Hoodie since March 24? See https://github.com/issues?q=org%3Ahoodiehq+author%3Acodyzu+is%3Aclosed Or am I missing something?

@sjnorth
Copy link

sjnorth commented Apr 19, 2016

Thanks for the work @finnp - very useful script! I will put it all together today

@finnp
Copy link
Member

finnp commented Apr 19, 2016

@gr2m Yeah that is correct. I was calculating that wrongly. I published 1.0.1. Also I replaced your summary in this issue with the fixed summary.

@sjnorth If there is anything you think could be improved about the summary let me know. You can comment or PR on this repository: https://github.com/finnp/github-shoutouts

@gr2m
Copy link
Member Author

gr2m commented Apr 19, 2016

@finnp are you sure the 1.0.1 release? I don’t see it: https://www.npmjs.com/package/github-shoutouts Also I don’t see new commits in https://github.com/finnp/github-shoutouts – could you double check it please?

@finnp
Copy link
Member

finnp commented Apr 19, 2016

@gr2m Nope but now I pushed and published 😄

@gr2m
Copy link
Member Author

gr2m commented Apr 19, 2016

Two suggestions / thoughts:

  1. Can we show the real name instead of the GitHub usernames? I think @sjnorth looked up the names for the shout out blog post: https://github.com/hoodiehq/hood.ie/pull/276/files
  2. @sjnorth gave a shout out to everyone who commented. I think it would be nice to differentiate, there is a risk that some comments were not helpful or even negative. For that it would be great to see the comments or at least the links, for a quick review?

These are not urgent, I can also make issues on the repo. Great work you two!

@sjnorth
Copy link

sjnorth commented Apr 19, 2016

@gr2m @finnp yes it would be a big help to see the comment content right away in the initial summary list. it would make the process much easier. can it be done?

@gr2m
Copy link
Member Author

gr2m commented Apr 19, 2016

@finnp maybe instead of an HTML output, we could do an HTML output with content (e.g. comment excerpts) that can be toggled?

@varjmes
Copy link
Contributor

varjmes commented Apr 19, 2016

Wow this is an amazing script. Also amazing to see I've done 62 pull requests to Hoodie! Better keep up that total :)

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 a pull request may close this issue.

4 participants