Stricter Tests

[tylerflex]

Tox tests now check for

• clean lint.
• clean diff from black

Blacked code using updated black, adds spacing around **

[momchil-flex]

So we gotta make sure everyone is using the same - or at least somewhat consistent - version of black? We should not this down somewhere explicitly.

[tylerflex]

https://github.com/flexcompute/tidy3d/blob/tyler/strict_tests/tests/requirements.txt#L2

[momchil-flex]

Seems like the checks don't pass?

[momchil-flex]

I think the only reason they don't pass is because of this tidy3d/components/monitor.py:159:18: W0612: Unused variable 'tind_end' (unused-variable). I'm a bit confused, seems like it runs the tests on the code state that would happen after a merge with develop, as this issue comes from the latest develop commit.

This is generally good to know actually, it's better if the tests are run that way.

[tylerflex]

yea this is a typo. Let me fix and see if the tests run again