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.
- Check our roundup of the best Linux distros
- Here are the best Linux laptops for running Linux
- Also take a look at the best laptops for programming
Torvalds is known to occasionally dish out such advice to fellow kernel developers, though he has toned down his tutelage of late.
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.
- Subscribe to Linux Format magazine for more Linux and open source goodness