OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

precommit: add `pyproject`'s hooks

Open Borda opened this issue 1 year ago • 3 comments

Reaction to #1411. Since there are no setup.cfg, but there are pyproject.toml configs, it may be worse to validate them...

cc: @rbren, @li-boxuan

Borda avatar Apr 27 '24 13:04 Borda

Mind merging https://github.com/OpenDevin/OpenDevin/pull/1411 to this PR? There might be a conflict.

li-boxuan avatar Apr 27 '24 17:04 li-boxuan

Mind merging #1411 to this PR? There might be a conflict.

seems I can't change the target branch across forked repo, once it was open

Borda avatar Apr 27 '24 19:04 Borda

Oh I meant by cherry-picking that commit to this branch. These changes are interconnected and might as well appear in the same PR. Just my personal opinion, though.

li-boxuan avatar Apr 27 '24 21:04 li-boxuan