-
Notifications
You must be signed in to change notification settings - Fork 525
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feature/velocity_controllers/joint_trajectory_controller rebased to indigo-devel #116
feature/velocity_controllers/joint_trajectory_controller rebased to indigo-devel #116
Conversation
Is it possible to merge this PR? |
Hey there. I was on holidays, hence the response latency. Could you please squash the four relevant commits into a single one?. After that we can merge. |
I have no experience in squashing stuff. How do you normally do this? I don't want to break anything... |
This is a nice way, by the book (have options not to squash some stuff): My personal preference (if you have a clean linear history to squash): In the optimal case squashing commits will leave you with a branch that has a single commit on top of the original repo. Squashing changes the history of the branch, so when you push it you will have to use "git push -f". Try squashing commits on a different branch first so that you won't endanger your PR contents. |
For this particular case, this is what I recommend:
In git, you can do all of this in a single interactive rebase (check out @bmagyar resources for the details of what's going on):
This will fire up a text editor with the four commits of this PR, starting with the oldest one.
|
…ature branch - feature provided by @davetcoleman
9df7530
to
4e812d8
Compare
@bmagyar @adolfo-rt I cleaned up this PR so that in only consists of the relevant commit. This should be it! |
+1 |
…ntTrajectoryController_indigo feature/velocity_controllers/joint_trajectory_controller rebased to indigo-devel
Yay! |
This is a rebased version of #109