-
Notifications
You must be signed in to change notification settings - Fork 18
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
Newsletter #11 #44
Newsletter #11 #44
Conversation
common-readme: I think it might make sense to drop this from the newsletter? It's not specific to IPFS and I think it's confusing to list both projects here. If you want to link to it within your repo though as a mention that'd be cool. |
What period of time does this weekly cover? I will mention various things once i know |
@jbenet The last couple of weeks; anything since April 25th. |
Added Contributors for this period (2015-04-25T00:00:01 to 2015-05-16T00:00:01) Note that I just pushed a small change to get_commits.py that will now use the github API to get the commit authors github login/name. See 4416c5d Edit: April 2015 was most certainly not the correct data range to use! This has now been fixed, thanks @dignifiedquire . |
* [@wking](https://github.com/wking) (W. Trevor King) | ||
* [@bmcorser](https://github.com/bmcorser) (bmcorser) | ||
* [@krl](https://github.com/krl) (kristoffer) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This list seems very incomplete. @nginnever @diasdavid and me (@dignifiedquire) have pushed a decent amout of code in the past weeks. Also krl is mentioned twice in this list
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, yes. Where are you very important people. Let me look. Thanks for paying attention!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh my goodness. i did a terrible thing. when I ran the tool, I used 2015, not 2016! A fix is incoming!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:D that would explain this list
thanks @eminence! |
Got another bug in |
@eminence I want to close this newsletter and start fresh with a new one. Can you pull out your commits from this PR and put them in their own? |
I had 3 commits in this PR, 2 of them related to the newsletter itself, and 1 was a tools change to get_commits.py. That tools change (4416c5d) is already in the master branch, so it's now safe to delete this PR |
e380fdd
to
77db928
Compare
Thanks @eminence. Rebased. This newsletter is very old at this point, and the next one should be started anew. I am going to close this and not try to send a retrospective newsletter for this two week session, but rather for the two months or so since the last one. |
First draft of IPFS Weekly #44
Renamed Weekly to Newsletter.
Added in all comments from #41.
Needs: