Hacker News new | ask | show | jobs
by xupybd 730 days ago
That's a lot of work. It really depends why they are doing it in the first place.
2 comments

That's the very basic job of being a maintainer. If you don't read and understand the code, you should just not merge it.

Nothing forces you to merge contributions. But if you do, do it right.

How big are the changes? Is this a huge new feature or some small fix?

My recomendation is to look at the code and secretly use Google Translate to see if they make sense. If they make sense, ask them to translate the messages, and then fix any typo of small gramar error before merging.

(Some mantainers like to ask to fix any minor error, but it's anoying for the contributor.)