recordexpungPDX icon indicating copy to clipboard operation
recordexpungPDX copied to clipboard

Add `pre-commit install` to make up

Open KentShikama opened this issue 4 years ago • 1 comments

This may involve documenting how to install pre-commit.

KentShikama avatar Jun 11 '20 19:06 KentShikama

Hi Kent, I've got a draft for the necessary changes to the Makefile, though I'm curious your thoughts on documentation. Currently this project states that it relies only on Docker and Docker Compose with no other native dependencies required. Including pre-commit is obviously an addition to this and I'm wondering how best to update the README. I'm thinking something like the following might work, though again would love to get an opinion on the overall voice and approach here.

Our dev environment is entirely containerized with Docker, with minimal native dependencies required.

The actual installation instructions should be trivial, thanks in advance for input on the above though!

jerrywardlow avatar Jun 12 '20 00:06 jerrywardlow