I was looking through various RCON tools and found this. Someone does not like commit messages.
At least it’s better than me messing up with the commit date :v
lol how did u do that?
Don’t commit near a black hole, apparently
what in the
very odd behaviour to say the least
that’s their personal project or I hope anyways
Not really programming horror. Just open source. They did something, are kind enough to make it open, they don’t care about commit messages so what?
It’s certainly not as if anyone is entitled to anything better. It’s just bad practice. I wouldn’t want people critiquing me based on the work I do for free either, though.
And that’s the issue I have with that. Imagine you are fabricio and you find this post. Next time you probably won’t make it public even if there is a remote chance, that it might help someone else.
I agree, the post shouldn’t have called him out directly by name. It should’ve been a post of just a screenshot with the commit messages, and not mention the repository or the person. It’s definitely not nice, and may even affect the person - I mean, if I was called out like that somewhere and seen people laughing at my practices, I wouldn’t be exactly happy.
On the other hand, I think that raising the issue in itself - FOSS repositories without commit messages - is a pretty good discussion that fits here well. It’s just a shame that it had to be at the expense of some FOSS developer, because it also derails the discussion into talking more about the guy and whether he can or cannot do it (such as your comments, which I agree with), instead of focusing on what are the issues with FOSS projects without commit messages in general. Because there are many arguments why it’s important, and a few arguments about why it shouldn’t matter, which makes for an interresting discussion about best practices and why they are important.
I think that there’s one important point to consider that may not be immediately obvious, when deciding about commit messages in FOSS project, even if you are not accepting contributions and just want to share your work for others - auditabilty.
7-zip has been receiving critique for this for a long time - not having commit messages makes it way harder to check what the actual changes were. Sure you can’t trust commit messages during an audit, but it makes it a lot easier - either you immediately notice that they are lying, or they are correct and will help you with understanding the change, so you can decide for yourself whether it’s safe.
Of course, the author is doing a lot of work for free, that he offers to others, so we have no right to blame him for it or demand he changes his approach. I’m grateful for any FOSS project, and demanding from someone directly that he’s doing it wrong and should do it some other way (or belittling him for it) isn’t OK. However, I’d probably be very careful when encountering a repository like this, and reconsider whether it’s worth adopting. Which is absolutely ok and I don’t blame the author for it in the slightest - it’s his repo and his work - but I also think that auditability may be something the author didn’t realize, and assuming his goal was to share his code with others i.e to build a portfolio, may affect his overall adoption rate. But it’s also ok if he simply doesn’t care about that.
But in general, if you’re making a FOSS project, I’d recommend sticking with good commit messages.
