Y
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
raverbashing
2585 days ago
Yeah no because this changes the code and does not merely complain
link
IanCal
2585 days ago
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.
link
raverbashing
2585 days ago
Yeah, true, though ideally you would do it before the commit so you wouldn't have one extra lint commit per code commit
link