Imagine this: you've been paged to investigate a production incident, and after some digging, you identify the commit with the breaking code. You d...
For further actions, you may consider blocking this person and/or reporting abuse
Cleaning up your local commit history before pushing is the norm in any half serious project. Happy to see you introduced the concept to so many new people!
However (and feel free to ignore):
Using the
git resetcommand for this purpose is like cutting your toenails with a lawnmower whilst blindfolded. This job would be so much faster, easier, and far less risky withgit rebaseinstead. No thumping headache or teetering on the edge of insanity afterwards either!Use fixup/amend commits to modify changes you've committed earlier (but have not yet pushed). These commits receive a special mark that Git will be looking for during a rebases with the
autosquashoption. Git will figure out how to get these changes in with the rest, and in such a way that no one will ever even know they happened.No matter how bad or embarrassing your brainfarts ever get, Git will be there to help you cover up, forget, and move on. A friend like none other :)
Another one of rebase's neat tricks is its
interactiveoption. It'll presents you with a list of your recent local commits and lets you rename/reorder/undo/or otherwise modify them as you see fit. All together in one go. Git will even warn you if your musical chairs session would result in conflicts - allowing you to abort, and puzzle some more to avoid unnecessary merge commits.Just like that you're done, no weird out-of-branch experiences, no worries about rewriting history, caught up that old Git again, got your commit in a state that's actually somewhat presentable, and all that within a fraction of the time.
Git Commit docs here: git-scm.com/docs/git-commit
Git Rebase docs here: git-scm.com/docs/git-rebase
Git grokking :)
git rebasecan save time in many situations, but it won’t help much if your rebase plan looks like this:Some real-world story…
I was debugging to find out why my Git deployment pipeline failed, so I had to make a lot of small tweaks to the pipeline script, commit, and push to see if the CI build is back working. Since frustration is rising, having quick feedback loop is very important, and so any extra cognitive load has to be reduced.
Composing a commit message required context switching between “problem-solving mode” and “reflecting-and-summarizing mode” which incurs significant load on my brain, so it has to be skipped. In the end my Git log looks like this (this is from an actual project):
In this case, to clean up the history,
git resetwould make it easier. I don’t think thatgit resetis dangerous or risky at all: It never touches the working directory, and if I messed up,git reflogalways has my back. (git reset --hard, on the other hand, is a whole another story.)This reached a state so far beyond a messy Git history, that it turned into a work of art. Linus Torvalds would be proud! 😅
I agree that the reset tool would have been a solid choice here. Comparing diffs/patches works well if you have only a few unknowns. Looks like you had slightly more to cover here..
My first thought too was "why not rebase?" but, and maybe I'm just doing it wrong, I've always had trouble if I want to change a commit, or if I want to put only some of the changes from one commit into another. I like this reset approach as an option at least to get my commit history the way I want.
If you start modifying commits during a rebase (which you likely will), then you run into the potential for conflicts, especially if you are reordering.
If you get into such situations and don't understand
$ git rebase --abort
Take smaller steps during the rebase. Move one commit (only a few commits back). Make only one change. Then rebase again for more changes.
Your bio checks out, good advice to take an incremental approach here!
Using fixup and amend commits won't eliminate these situations completely, but they'll definitely help towards keeping track of the commits without having to dive into their individual diffs.
Author here – hello! 👋
I didn't mention it the article, but I also use rebasing quite often! Especially to keep my branch up to date. You can also use interactive rebasing to achieve the same goal, it really just comes down to what works for you. Using
rebase -iis helpful if you know that at least some of your commits can be left in tact, or don't need as much modifying. In fact, once you get comfortable enough with it, usingrebase -ican be even faster than doing a fullreset, because you're not having to re-write as many commits. 💪That said, staying focused on development while also thinking about "Wait - should this work be in a separate commit?" requires some mental task-switching, which can be taxing on your working memory if you have an overactive neurodivergent brain like I do. I've found that the method described in the article frees up so much mental bandwidth because I can hyperfocus on one goal at a time, be that writing code, or organizing my commits.
If the goal is ultimately to have a sequence of clear, concise commits that are easy to review (or revert), then it really doesn't matter which method you choose. :)
Absolutely do what works best for you. As long as your pushed changes fit the branching strategy you're golden. Well, and not waste too much time putting back together the history you *just fixed* of course 😉
I'll try to get you an example sometime next week. You might not sell on it, but at least you'll know how it works!
Totally agree with that comment.
Notice some good stuff you may also use:
git reset HEAD^in order to reset last commit (useful to split commit in the middle of a rebase). Notice that some IDE will provide built-in tooling for that.git push --force-with-leaseinstead of--force; that's gonna check if anyone pushed in the meantime (it should not, but I consider it as a good practice since it will warn you)I agree. Rebase is better option!
Very nice post. I use this process as well.
The only step I sometimes do differently is: Before resetting my feature branch, I copy it. I branch off the latest commit of my feature branch into a "temp" branch. Then I follow your steps.
However, if I mess up my afterwards, I can just reset my feature branch again and fast-forward merge my temp branch into my feature branch, bringing it back to the original commits before I reset anything. This means I don't have to mess around with the reflog, which can be quite complicated.
Thanks and keep up the good work 👍
Instead of copying, you could make an empty commit (and tag it, maybe), so you would have the exact point-in-time where to go back if needed. Could be something like:
Your branch-copy approach is interesting as well, because you can just checkout to the backup branch and go back to where you wanted.
nice write up, it is a lot of reading though... lol. I follow a very similar process myself, adhering to the traditional "commit small, commit often" concept as closely as possible.
if I end up resolving unrelated bugs while working on a feature branch, I normally do the following (assuming a "single main branch" workflow, using a "fast-forward merge only" strategy with no merge commits):
general-fixesfrommasterfeature-xfeature-xontogeneral-fixesto verify thingsgeneral-fixesintomasterfeature-x..quite often this happens multiple times a day, so it doesn't take long for it to become habitual.
..and I apologise ahead of time, but I just need to rant a little bit here: I really, really, really wish I could work with people that understand and appreciate Git. :(
one of my work colleagues is of the opinion that rebasing branches after cherry picking commits actually creates duplicate commits.. but he also manually synchronises entire folders of code between multiple projects, and refuses to use any code formatting tools.. so, meh. :insert_epic_facepalm:
I really understand your rant, since I always went through this. But I try to advocate for git and better git flows as much as possible.
I love to commit often, too. I am a branch addict and create sub branches for branches I'm working on, only to (try to) better organize my job. But this
git resetapproach really seems to work for me. Can't wait to try this soon!As a branch addict, I always create WIP branches only to check them out later, some specific file that worked and so on. Again, with this
git resetapproach I believe this will not be necessary anymore.I am learning to code and I think this post will be a great solution to future projects I am apart off, Thanks
Glad you found this helpful!
If the goal is to end up with a series of well organized, self-contained, logically sensible commits, doing it while you are coding them, with fresh memory of what is for what, is still better than accumulating it all until the end, starting over and trying to sort through a big pile of changes.
Remember when you are wrapping up a feature branch, you have just gone through a lot of stress of coding, and are likely under a lot of pressure to deliver. At that point, I doubt many people would have the appetite to sort the 74 diff blocks in that one file into several separated commits. They would just lump them all up into one big commit.
This is a really interesting approach which I hadn’t considered. Thank you. Reviewing some responses it seems that get shares a problem found in other tool advocate camps of righteous zealots suggesting there is one single right way to do things. One of my favorite things about get is there’s so many different ways to do things permitted by the many different commands available. This flow that you proposed is interesting and I could see it being useful but I don’t have to automatically jump on the bandwagon of saying this is how it ought to be done, nor do I think that you were suggesting so. Similar to the many commands that git offers, this workflow is one of many workflows that might make sense at any given point along side and with others at different times, in different branches, and in different projects.
Hi Annie
Thank you for the short nice article. It is very helpful.
One point you must do to avoid misunderstanding and keeping your article as simple as it is:
keep the name of the branch as it is.
At the beginnging of your article you create my-featrue-branch
But later you when check the status after the reset, the branch name is different:
This is confusing and will make newbies more unconfident.
It would be a pitty if unexperienced people get conused
Thank you for sharing :-)
According to my, well-working, standards, my
git-workflowis as follows ( lets takerepoAas my project's repo, andlocalas local laptop/workplatform; )Make changes, than
Terrible idea.
It can be used as "last chance" when you totally messed up your commits but you should never use it for normal process.
Just commit often (it's your branch after all) and use "git squash" or "git rebase -i" before creating pull request.
If you made mistake during squash/rebase – "git reflog -> git reset" and try again.
We squash merge PRs
why squashing merge PRs? Simply merging is perfectly fine.
Probably because he doesn't want to "pollute" the destination branch with all the detailed and sometimes maybe incomplete commits. So, only one commit containing all the previous commits as comments may be better for his needs :-)
I agree, that this approach has its place into the world as well as rebasing. In the previous company we had a huge repository with a lot of projects in it(yeah - "brilliant" idea, because the DevOps guys and gals couldn't fix their Jenkins jobs) and for the sake of easier traversing the history if needed we have agreed that a single commit should be for resolving a single story or fixing a single bug. And actually the whole process was build around - you are working on a single thing. Then in our case squashing multiple commits(rebasing doesn't byte too hars when you play with it several times) into a single one was the choice.
Oh, another neat command is pull --rebase which can get the latest state from the branch you want and push your commits infront of it. If done before merging - you will see(and hopefully properly resolve) all collisions and your changes will be on top of the main/dev/smthg else branch.
This doesn't sound like it'll work.
If you fixed bugs while working on your feature, those bugs are most likely in the same files you were working on.
So you will end up committing unrelated changes this way. Honestly, if this is your strategy, you might as well not commit anything at all, and just commit everything when it's done. (and to be clear, I am not recommending that.)
Having that unrelated bugfix in a separate commit (like you probably had in the first place) you can cherry pick it to the main branch or a separate PR (depending on your workflow) before submitting your feature PR.
It's probably better to work on your commit discipline. Commit in small increments, commit when you're in a working state only, diff and review each change before you commit, and always use the commit log to describe "why" - the "what" is usually obvious from the diff.
That's exactly how I work except for I tend to create what you describe as final commits along the way. With this in mind I can avoid to lose something on the way and produce working commits without including/excluding things that can become quite combersome to manage and group at the end.
That's a very interesting point of view. I always had thought how difficult it is to structure a new project in your head before writing a single line. My head prefer to start writing and the structure simply shows up, surely because my lack of experience.
This method will help me a lot with that! thank you for sharing Annie
it's an opinion but don't u think it's a longer way of doing this. instead just squash the commits when u merge the PR. it will consolidate the commits into one single unit. ever u need to revert u can do it with just a single commit.
This is exactly what we've been doing for years, can confirm it looks nice and works great.
Seems to be a nice approach to make cleaner commits :)
I’ll have to try this because I’ve never used reset that way. I suspect, like many other commenters, that it’s very unlikely my bug fix can be separated from the other changes I’ve already made.
This feels like a problem best solved without technology: write your thoughts on a post-it and come back to it later. I’ve pair programmed with some XP-experts and they were all very good at this.
The solution to these two problems I’ve found, lies not with git savvy, but with a whole different philosophy to your code in master.
Everything in master goes to PROD.
and this is only possible with a feature toggle infrastructure. When you have this, a PR that is WIP can be merged with little to no side effects. This then leads to smaller PRs, smaller PR review burden, and much simpler git usage.
This isn't a bad approach to dealing with the problem you describe - something broken in a different commit. I squash commits into a single one when merging to main so that if I need to revert, I'm reverting all of my work on that feature and not just one of several commits. I'm also in the rebase camp on this.
A great and simple guide to follow !
That's great article, I haven't test this workflow but my first impression is the use of "git reset origin/main" Assuming if your go-workers push number of commits to the origin/main repository. You pull often to update your local repository and avoid conflict.
I'm wondering if have multiple pull from "origin/main" during your work on your feature branch and then you run the "git reset origin/main" at the end of your work. Does it still have the same result you expected?
I haven't test, but I would like to see how your workflow works.
This is a workflow I hadn't considered before. Typically, I'll go with an interactive rebase before preparing a PR, and other times I'll do an initial commit and just amend it every time I make changes. The latter has some pretty obvious downsides, unless you're working with a relatively small change. I'm going to give this a shot on my next feature branch and see how it goes. Thanks for sharing your wisdom!
Ever since I became se3mi-retired and a 'hobbyist developer' I've gotten rather lazy as far as git is concerned. I really need a kick and get things sorted out.
And then there is git-scm.com/book/en/v2/Git-Tools-I... if you need more finegrained options which things add to which commits.
Pingback :
This article was curated as a part of #42nd Issue of Software Testing Notes Newsletter.
To add onto the people saying to use fixup and squash, here is how to do it quickly and efficiently.
dev.to/kwstannard/how-to-plan-an-i...
Nice share 🔥
A bit of a self plug, but i think it might help people
you can use ugit to undo git commands
github.com/Bhupesh-V/ugit
This is definitely a tool to use for making better commits. I attempted to answer why go to all of this trouble.
Git is a Communication tool
Jesse Phillips ・ Dec 12 '18 ・ 2 min read