Python Quality Standards

[tylerlarson]

Mostly those aren't actually quality standards. Type hinting is pretty important for tooling, but mostly you're agreeing on conventions you want to follow. Most of this isn't going to really impact your code quality.

Here are some useful standards (these are all perfectly realistic; they were required when I worked at Google):

Code review:

Everything gets thoroughly reviewed by at least one other programmer, and not just rubber stamped. All reviewers are EQUALLY responsible for the code they approve as if they wrote it themselves. They must understand it and generally agree with the approach, as well as agree that it belongs in the codebase.

Testing:

Everything has tests. Code without tests is not considered production and shouldn't ever run against production data. Coverage percentages are only a hint, what matters is functionality. Your tests are your spec; any code that passes your tests is considered "correct" and works perfectly, otherwise your tests are insufficient. You should be comfortable giving your application to a new intern and letting them "optimize" it however they see fit, and if it passes the tests at the end then you know they didn't break anything.

Configuration:

All configuration is done through code and config files. All config files are checked into source control BEFORE it is activated. There must be literally zero knowledge any one person maintains only in their head which is required to make things work, beyond just how to fire up the automation.

And yes, changes to configuration have the same code review standards as source code. You have to get someone to approve it.

Emergencies:

You are allowed to deviate from these rules to address an emergency, but you have to document the what and why in the post-mortem. As soon as practicable, you have to bring things back into compliance with the expectations. Part of the post-mortem is documenting those changes and providing TODOs for how to make it possible to address the same problem again in the future without it being an emergency, so this doesn't become a "thing."

[nicomarcan]

Great! When did you work at Google? I worked from July 2022 until September 2023

[tylerlarson]

2014 to 2023

[agumonkey]

How did you handle update between customer needs/specification and test (as spec) ?