Hacker News new | ask | show | jobs
by im_new_here 2585 days ago
CI runs a linter and fails the build if the output is non-empty. PRs can't be merged unless the build passes on the branch.
1 comments

Yeah no because this changes the code and does not merely complain
One, that shouldn't matter on CI anyway. Two, you can tell black to just print the difference with --diff and tell it to just check the files with --check.
Yeah, true, though ideally you would do it before the commit so you wouldn't have one extra lint commit per code commit