- pre-commit doesn't necessarily exist as a system package (i.e. Debian buster),
but more commonly as a Python package, so install via pip.
Then, mention to update PATH if need be.
- install local swh packages as a 2-step process to avoid errors
Differential D3867
Adapt developer setup instructions as of sept 2020 tenma on Sep 2 2020, 6:31 PM. Authored by
Details
but more commonly as a Python package, so install via pip.
Diff Detail
Event TimelineComment Actions Thanks, good catch! One comment, though:
Comment Actions The idea is to install it outside of a venv, to access it like the system package.
I don't think it needs to be py3 because instructions mentioned the Debian package which is py2 I think. EDIT: pre-commit seems to be used as a Python library in swh Git hooks, so it definitely shouldn't be installed as a system package Comment Actions Changed to install pre-commit as a python package in venv. |