Attention is currently required from: osmith.

Patch set 1:Code-Review +1

View Change

1 comment:

• File _testenv/testenv/__init__.py:

  • Patch Set #1, Line 215: packages

    Nitpick: package(s) suggests that you can specify more than one package, like `--install-packages "foo bar zoo"`, but the `metavar` is `PACKAGE`. Maybe call it `--install-package` (without the 's') to avoid confusion? Alternatively, maybe change to use `type=str` and then do `testenv.args.install_packages.split(' ')`?

To view, visit change 41215. To unsubscribe, or for help writing mail filters, visit settings.