This rebase really threw me off, I think we shouldn't include this.

I did this:
1. make some commits to a submodule, merge a branch inside the submodule
2. feature pull on the parent repo.

And I ended with with brand new commits in my submodule, it did a rebase got rid of my merge, rebasing the commits inline instead of merging...  I feel like parent commands shouldn't create new commits in the submodule, only checkout a different HEAD.

What do you think @xiongchiamiov ?