Review Request #10179 — Created Sept. 25, 2018 and submitted
Our prepare-dev.py was doing quite a few things that weren't best
- Global mutable state in the form of
- Using the deprecated
optparsemodule instead of
- No docstrings anywhere.
These issues have all been addressed. Where we were previously using
global mutable state we now pass our state as an argument to the
function that requires it (
create_settings). Our argument parser has
been updated to use
argparse.ArgumentParser. The module and all
functions now have docstrings that meet our documentation guidelines.
Additionally, I cleaned up control flow a bit in
main() where things
were running in a
try..except that didn't need to.
prepare-dev.pyand it ran correctly.
E303 too many blank lines (2)
It's kind of weird to have this be this late. I know we had a few people run into issues ...
I think this can just be "Parse the command-line arguments". Creating an argument parser is just an implementation detail that ...