# PRs and granularity

**URL:** <https://meta.discourse.org/t/prs-and-granularity/49162>\
**Category:** Development\
**Created:** [24.Август.2016 19:02:31 UTC](https://meta.discourse.org/t/prs-and-granularity/49162 "2016-08-24T19:02:31Z")\
**Posts on this page:** 1\
**Showing post:** 7

<div class="post-metadata">

**Author:** ![pfaffman](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/pfaffman/32/120154_2.png) [@pfaffman](https://meta.discourse.org/u/pfaffman)\
**Post date:** [07.Сентябрь.2016 00:44:26 UTC](https://meta.discourse.org/t/prs-and-granularity/49162/7 "2016-09-07T00:44:26Z")

</div>

I should arguably post this at Stack Exchange, but this community’s practices are those that I’m interested in following and I already started this thread. . .

OK, so I spent several hours today trying to do something like the above on 30 minutes of code changes in `discourse-setup`. I figured out that I can get Emacs to do what you’re doing with `git add -p` and managed to submit [let discourse-setup install docker](https://github.com/discourse/discourse_docker/pull/287) and [make memory tests work](https://github.com/discourse/discourse_docker/pull/286).

If I were to do that again, would the recommended course be to do this?

1. `git checkout master`

- do the changes for the First Thing
- `git checkout -b first-thing`
- `git push master first-thing`
- `check checkout master`
- do changes for second thing
- `git checkout -b second-thing`
- `git push master second-thing`

And I suppose if I were clever, I could then merge first-thing and second-thing in my own fork if I wanted.

---

_[View the full topic](https://meta.discourse.org/t/prs-and-granularity/49162)._
