- The most common workflow complaint I've seen recently is something like "how do I use Differential with a branch full of random code that me and several other developers all commit to"? There are some okay answers ("commandeer") but I think the best answer is "don't do that". Add a document explaining how development works at Facebook (and many other companies) without the use of feature branching, why it's better, and how you can lay the technical groundwork you need to to stop doing this.
- Add a general "smaller commits are better" and "your commit messsage should provide context" document.
- Minor updates to other stuff as my understanding of Mercurial has been refined.
Details
Details
- Reviewers
vrana schrockn btrahan - Commits
- Restricted Diffusion Commit
rP49205f977cca: Add/update some documentation on branches and reviewable code
Generated and read documentation.
Diff Detail
Diff Detail
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
| src/docs/flavortext/writing_reviewable_code.diviner | ||
|---|---|---|
| 75–76 | (I thought this was a pretty cool stat, although I'm not sure how well we're doing relative to other projects.) | |
Comment Actions
this is awesome and i love the median checkin stat.
I'm wiped out (just closed out the strata apartment, which meant moving shit to sister Melissa's place up many flights in the mission and sister-in-law's stuff to my house). I want to give this a good read / feedback since I think its a pretty important / awesome article. That'll maybe be later today if I recoup well and if not tomorrow.