Don't use GitHub to merge commits, suggests Torvalds

Developers
(Image credit: Shutterstock)

Linux creator Linus Torvalds isn’t a fan of how GitHub’s web interface merges code, which he believes creates “absolutely useless garbage merges.” 

The commentary follows a pull request from Paragon Software who nudged Torvalds to include its driver code for the Windows NTFS filesystem into the 5.15 kernel.

“... I notice that you have a github merge commit in there. That's another of those things that I *really* don't want to see - github creates absolutely useless garbage merges, and you should never ever use the github interfaces to merge anything,” wrote Torvalds in response to Paragon’s pull request, before illustrating the shortcomings of making commits via GitHub’s web interface.

Torvalds is known to occasionally dish out such advice to fellow kernel developers, though he has toned down his tutelage of late.

Proper commitment

One of the shortcomings Torvalds highlighted are GitHub’s concise, factually correct, but functionally useless, commit messages. 

For instance, GitHub’s commit message for Paragon’s merge read “Merge branch 'torvalds:master' into master”, which didn’t impress Torvalds one bit. 

“github is a perfectly fine hosting site, and it does a number of other things well too, but merges is not one of those things,” Torvalds shared. 

He then went on to explain that he prefers kernel merges to be properly documented with details about what is being merged and why, along with proper and accurate details about the committer, which he believes isn’t one of the strongest suit of GitHub.

Although he let it slide this time, Torvalds suggests Paragon do future merges from the command-line.

“...for continued development you need to do things properly. That means doing merges from the command line, not using the entirely broken github web interface,” he concluded.

TOPICS
Mayank Sharma

With almost two decades of writing and reporting on Linux, Mayank Sharma would like everyone to think he’s TechRadar Pro’s expert on the topic. Of course, he’s just as interested in other computing topics, particularly cybersecurity, cloud, containers, and coding.

Read more
Shadowed hands on a digital background reaching for a login prompt.
This worrying Git flaw could lead to users leaking credentials
An abstract image of digital security.
Hundreds of GitHub repositories hijacked to trick users into downloading malware
An image of network security icons for a network encircling a digital blue earth.
Wireshark to the cloud! Founders tell us about open source, Windows v Linux, and more
Google Chrome
Linux Foundation brings together top browser makers for more "open" approach
MacBook Pro with Asahi Linux logo
Head of top Apple Mac Linux distro resigns, citing burnout and slamming Linus Torvalds leadership
Facebook on laptop
Facebook is blocking Linux topics and channels with no apparent reason
Latest in Software & Services
TinEye website
I like this reverse image search service the most
A person in a wheelchair working at a computer.
Here’s a free way to find long lost relatives and friends
A white woman with long brown hair in a ponytail looks down at her computer in a distressed manner. She is holding her forehead with one hand and a credit card with the other
This people search finder covers all the bases, but it's not perfect
That's Them home page
Is That's Them worth it? My honest review
woman listening to computer
AWS vs Azure: choosing the right platform to maximize your company's investment
A person at a desktop computer working on spreadsheet tables.
Trello vs Jira: which project management solution is best for you?
Latest in News
An image of Pro-Ject's Flatten it closed and opened
Pro-Ject’s new vinyl flattener will fix any warped LPs you inadvertently buy on Record Store Day
EA Sports F1 25 promotional image featuring drivers Oscar Piastri, Carlos Sainz and Oliver Bearman.
F1 25 has been officially announced, with this year's entry marking a return for Braking Point and a 'significant overhaul' for My Team mode
Garmin clippd integration
Garmin's golf watches just got a big software integration upgrade to help you improve your game
Robert Downey Jr reveals himself as Doctor Doom to a delighted crowd at San Diego Comic-Con 2024
Marvel is currently revealing the full cast for Avengers: Doomsday, and I think it's going to be a long-winded announcement
Samsung QN90F on yellow background
Samsung announces US prices for its 2025 mini-LED TV lineup, and it’s good and bad news
Nintendo Switch Lite
Forget the Nintendo Switch 2, the original Switch is getting one last hurrah in a surprise Nintendo Direct tomorrow