-
Notifications
You must be signed in to change notification settings - Fork 28
Review Checklist
Anthony Miyaguchi edited this page Sep 7, 2017
·
1 revision
- Are all the arguments using
@click.option? - Are all required options using
required=True? - Is the command registered with the command line interface?
- Is
utils.mozetlimported into the scope? - Are the variables wrapped using
mozetl_envvar? - Does the name of the command match up with the
mozetl.cli? - Do the environment variable names match up with the option names
- Are all required options accounted for?