I'm a web sysop and support engineer. My skills are mainly in back-end: Java, Linux, Python, PostgreSQL, Git, and GitLab. Currently I'm learning front-end skills: JavaScript, and Ruby.
I feel it's better to make a computer do code linting, than to do this in a PR review. Nobody minds when a computer picks nits, but when people do, it's taken personally 😉
The review should focus on the logic, that the PR makes sense, and then run it though a linter for code style.
Bonus points to the reviewer for lint instructions or offer an editor.conf for the project.
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
I feel it's better to make a computer do code linting, than to do this in a PR review. Nobody minds when a computer picks nits, but when people do, it's taken personally 😉
The review should focus on the logic, that the PR makes sense, and then run it though a linter for code style.
Bonus points to the reviewer for lint instructions or offer an editor.conf for the project.