Add tests

Check variable names

Code order makes sense

It has docstrings or types

Error handling and logs

Functions have a single responsibility

Functions avoid side effects when possible

Documentation is updated