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.
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.