Add check-manifest and clang-format to pre-commit setup (removing from make)
Remove intermediary make commands
Clean up Makefile and C files found during linting
Configure pre-push hook to run in CI
Remove extra requirements absorbed into pre-commit
Testing performed
Reinstalled the hooks to allow for pre-commit and pre-push hooks. Running pre-commit run --all-files runs in the pre-commit stage so all but check-manifest. Running pre-commit run --all-files --hook-stage pre-push only runs check-manifest.
Issue number of the reported bug or feature request: closes https://github.com/bloomberg/memray/issues/360
Describe your changes
check-manifest
andclang-format
to pre-commit setup (removing frommake
)make
commandsMakefile
and C files found during lintingTesting performed Reinstalled the hooks to allow for pre-commit and pre-push hooks. Running
pre-commit run --all-files
runs in the pre-commit stage so all butcheck-manifest
. Runningpre-commit run --all-files --hook-stage pre-push
only runscheck-manifest
.