Software Code Review

@jamesasmith Please use this issue to discuss and document your code review. Some guidelines for the code review can be found here:types-software-review

Note we do use a wide array of pre-commit hooks when committing code that include black, flake8, and prettier, among others that insure consistent code formatting. There is an automatic testing suite run against merge requests that include testing, coverage, safety, and pre-commit. These align with many of the suggestions in the code review.

@alexprescott_email.arizona.edu and @nfoks1 Please add any additional thoughts if you have them.

This process is new to us all (I think) so please don't hesitate to ask questions.

Thank you!