Skip to content

Review Checklist

Anthony Miyaguchi edited this page Sep 7, 2017 · 1 revision

Review Checklist

mozetl

  • Are all the arguments using @click.option?
  • Are all required options using required=True?
  • Is the command registered with the command line interface?

airflow

  • Is utils.mozetl imported 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?

Clone this wiki locally