Format code with black and isort

What does this MR do and why?

To ensure consistent coding styles and imports across the repo, we use black and isort.

• Add some Makefile utilities to run lint and test locally
  • make lint
  • make test

I have restrained from adding this check on CI and running the format across the code base since it will create a lot of conflicts for existing merge requests. I will add some follow-up MRs to:

• Lint only changed files
• Fix existing lint violations incrementally