As described in the pull template: " Each component has a single commit (if not, squash them into one commit)" - so yes this is generally good. It makes review easier, we also don't want to include your local development "history" in our master branch.