Yep, I understand but disagree. Maybe it comes from working with so many ESL coders, but I’ll happily accept typo corrections because it’s not always obvious what words should be if you’re not steeped in the culture.
If you’re a larger project, you can see a ton of these from people hoping to land a commit to put “contributor to X” on a resume somewhere. Those add up and are really distracting and possibly automated. They waste everyone’s time, especially if they spawn a bunch of conversion like this did.
If you’re a smaller project, it doesn’t matter as much. I work with ESL coders too, so I get it (1/4 of my office is ESL immigrants, and ~2/3 of the broader team is ESL). I fix comments all the time, I just include them with other changes.
So it depends. But in general, a high profile project should reject this noise to discourage this behavior.
Yep, I understand but disagree. Maybe it comes from working with so many ESL coders, but I’ll happily accept typo corrections because it’s not always obvious what words should be if you’re not steeped in the culture.
It really depends on the project.
If you’re a larger project, you can see a ton of these from people hoping to land a commit to put “contributor to X” on a resume somewhere. Those add up and are really distracting and possibly automated. They waste everyone’s time, especially if they spawn a bunch of conversion like this did.
If you’re a smaller project, it doesn’t matter as much. I work with ESL coders too, so I get it (1/4 of my office is ESL immigrants, and ~2/3 of the broader team is ESL). I fix comments all the time, I just include them with other changes.
So it depends. But in general, a high profile project should reject this noise to discourage this behavior.