When you start programming, you’ll immediately hear that you should learn Git. Git is an essential tool in our industry, and it’s used for managing the codebase in teams of all sizes. What are the aspects of working with Git that can surprise you once you start working in a team? The remote repository will be busy with other developers’ changes. Separate branches are very useful for doing code reviews and catching bugs before they get merged into the stable branch.
Companies Mentioned
When you start programming, you’ll immediately hear that you should learn Git. That’s good advice. Git is an essential tool in our industry — it’s the most common version control system, and it’s used for managing the codebase in teams of all sizes.
As a person learning to program, you will experience only one part of Git : version control for a solo developer. When you are alone on the project, your daily Git usage can be limited to:
git add,
git commit,
git push
You could experiment with different branches, but this is mostly it. What are the aspects of working with Git that can surprise you once you start working in a team?
Busy remote
First and foremost: the remote repository—origin—will be busy with other developers’ changes. You will not only push your changes, but you will need to get the changes of others too. This will require you to use other Git commands, such as:
git fetch—To get changes from a remote repository, without changing the local working copy
git rebase—To move the whole branch from the place where it started and reapply the changes on top of another commit. This changes the branch history and can cause headaches in case of conflicts, but it allows for keeping history linear.
git merge—Joins two branches. Doing this keeps branch history intact and generates fewer conflicts, but it produces a git tree that is more complicated to follow.
Another command that is often used is git pull. Depending on your settings, it’s just a combination of:
git fetch and git merge—the default one, or
git fetch and `git rebase—one you can switch to via setting:
$ git config --global pull.rebase true
Lost? Catch up here
If you are a bit lost with the commands I mention above, you can catch up with —a great, free course that explains those commands with a visually pleasant interface:
Branches
Branches are one of the basic concepts in Git. On a technical level, they are just references to commits, but they are used by teams to organize changes in the repository. One of the common patterns is to have one main branch—often called main, master, or development—which is more or less stable, and new work is done in branches.
Thanks to branches, you can have multiple people working on different features in parallel. As each feature becomes stable, they are merged into the main branch, and the project moves forward.
You can use branches in a solo project, but usually, there is not that much need for it. At most, you can use branches to do some experiments with code and keep changes for picking back up later.
Code reviews
Separate branches are very useful for doing code review. In well-organized teams, every change is read by some other developer besides the author of the change. The goals of doing this:
catching bugs before they get merged into the stable branch
keeping another person in the team up to date with changes
letting developers discuss technical details of implementation
getting new colleagues up to speed with the team’s guidelines
As a new developer, code reviews are a great way to speed up your learning. And for people on all levels, it’s a great way of keeping code consistent and making sure people can share workloads when needed.
Code review is typically a feature offered by the Git hosting service. Depending on the git hosting provider, this process may be called:
pull requests—often abbreviated as PRs, on GitHub and many others,
merge requests—or MRs, on GitLab
Conflicts
When you have many people changing the same codebase, there will be some changes that Git cannot resolve automatically. In those cases, some developers will need to manually integrate changes in one of the branches. When you work in a team, at some point you will need to learn how to do it yourself.
There are some tricks to minimize risk of conflicts:
using tools to ,
merging branches fast
trying to avoid making overlapping changes at the same time as other colleagues
What other team members care about
When you join an established programming team for the first time, you may be surprised with how much attention other people pay to what happens in the Git repository. This can be confusing at first, but there is a good reason to care about how the Git history looks. In a programming career, you often have to investigate changes in the codebase done years ago made by people you’ve never met. Thus, having a clear Git history makes a lot of difference in those cases.
Commit messages
A about patterns for Git messages. You even have projects to validate automatically. It would be an understatement to say that people care about how messages are written.
As an example, I provided feedback at work about:
the length of the first line of the message
the possibility of making commit message span many lines
editing work-in-progress messages into one consistent commit message while squashing many commits into one
the place where the verb should appear in the message
grammatical “person” (e.g., first vs. third person) and tense used in the message
All that can be a bit shocking when it happens to you for the first time.
New lines at the end of the files
Unix has a convention of ending text files with a new line—it works nice when tools like cat display many files one after another. Most text editors don’t display this newline, but they keep it in the file. If your editor removes the line, Git recognize it as a change and shows the change as:
+++ b/hello
@@ -0,0 +1 @@
+Hello
\ No newline at end of file
If you add it to the repository, it’s very likely someone will ask you to add the new line there; and change the configuration of your editor.
Mixing unrelated changes in one commit
Good commits are atomic commits: commits that contain all the code changes related to change that is supposed to happen in the project, and nothing besides that. So, for example, mixing some unrelated changes (fixing the login logic and the color of the button) would be breaking this rule.
Why is it important? If you ever need to revert some of the changes, you wouldn’t want to revert more than necessary. So in our example, you don’t need to revert the button's color just because the login logic had to go back to what was there before.
In a team environment, it’s often the more senior people doing the reverts—and they will insist on juniors to commit changes in a way that it will be to do the cleanup if necessary.
Another common temptation is to mix code style changes with meaningful changes in the logic. This can cause the problems discussed above, plus it makes it very likely to cause conflicts.
How your branch looks
The person who does a code review of your changes can have no idea about what you are doing and why. The easiest way to get them up to speed is to have your branch tell a story—for example, to have multiple commits with messages like this:
do some preparation refactoring
resolve a minor issue in the logic
implement the main change in the logic
fix related styling issue
With a branch history, it’s effortless to understand why each of the changes was made.
How branches are named
Some teams have branch naming patterns that you will be expected to follow. Things I’ve seen in projects:
<ticket-reference>
<name>/<ticket-reference>
[feature|bugfix|doc]/[description]
Those things help a lot if you have a project with many people, and it continues for many years.
Keep out secrets
You shouldn’t publish your passwords or any other types of secrets. If you add an AWS key, hosting password, ssh key, or anything else to the repository and push it outside, then you should assume this secret is compromised, and it should be recreated. It would be a headache in a private project, but in companies it can be both dangerous and expensive — often there are many integrations that depend on a given secret, and it’s not always documented.
Only removing the value from the repository, or even its history, is not enough — not from the security point of view.
Keep out local configuration
There is no need for us all to see the local settings for your code editor, temporary files created by your operating system, or any other thing not related to the project. You should create .gitignore files to keep those things away from the repository. I covered it in .
Git workflows
Git is a powerful tool that gives you plenty of options for use in a project. To put some structure to all those possibilities, there are many Git workflows—recipes on how branches should be used in teams. You can find a nice summary of some of those approaches in .
Some developers have strong opinions or preferences regarding workflows. As a beginner, or simply as a more practical developer, you don’t have to care that much about them—just use the workflow of your team as a guideline for how things should be done.
Summary
Git is a programming tool that solves technical issues. However, it takes on a social aspect as soon as you start working within a team. What surprised you when you started using Git with other people?
Also Published