Suggestion: add git-hook hook for pre-commit about evals

[jonathanagustin]

I think pre-commit install will install it as an actual git-hook, but I see your point that it is not committed into the repo by default.

There may be some intention or concern to not do so because the comments mention that it is slightly dangerous:

evals/.pre-commit-config.yaml

Lines 16 to 29 in f118fca

	# this is slightly dangerous because python imports have side effects
	# and this tool removes unused imports, which may be providing
	# necessary side effects for the code to run
	- repo: https://github.com/PyCQA/autoflake
	rev: v1.6.1
	hooks:
	- id: autoflake
	args:
	- "--in-place"
	- "--expand-star-imports"
	- "--remove-duplicate-keys"
	- "--remove-unused-variables"
	- "--remove-all-unused-imports"
	exclude: "evals/__init__.py"

A manual run adds a kind of self-check to confirm the existence of unused-in-file imports. I see this visual check as a good thing in library imports like __init__.py (which is excluded in the code above) where the intention is to expose functionality but not necessarily use them within the file.