We (I work on addons.mozilla.org, too) try to keep the commits separate so it's easier to review. For example, if you get a review, and then have to make a few

We (I work on addons.mozilla.org, too) try to keep the commits separate so it's easier to review. For example, if you get a review, and then have to make a few changes and get a followup review, it's easier for someone to be able to see just the new changes. (They can also view all changes, by using the compare view.) Then, before pushing to master, everything is combined.