search

wics-uw / website

The code that runs http://wics.uwaterloo.ca/
Other
9 stars 49 forks source link

Update F15-event-pinterest-tech-talk.md #114

Closed ZiweiZhao closed 9 years ago

ehashman commented 9 years ago

Can we hold off on merging this until we have the names of the Pinterest engineers added? No point in having to deploy this twice.

ehashman commented 9 years ago

Cool, we have the names now. Thanls for taking the time to update this, Ziwei! I'm just going to merge this.

Also, @ZiweiZhao, in the future I ask that you check out my git style guide in the readme. In particular, your commit names do not convey any new information (it's clear you are updating that file from the commit, but what did you update?), and you should probably be rebasing your small commits into larger ones. We can talk about how to do this and possibly pair if you ping me on IRC.