Add CI configuration files #109
Conversation
| phases: | ||
| build: | ||
| commands: | ||
| - pip install --upgrade pip --quiet |
There was a problem hiding this comment.
quiet hides useful debug info
There was a problem hiding this comment.
I carefully try to keep only useful debug information. I did not consider upgrading pip useful. I am happy to remove if you have any specific considerations.
There was a problem hiding this comment.
without --quiet, pip shows versions of things it is installing, and whether it is reusing a package already installed. useless almost all the time, essential when there are are version conflicts or errors related to unexpected version change.
| commands: | ||
| - pip install --upgrade pip --quiet | ||
| - pip install tox --quiet | ||
| - tox -e ${TOX_ENVLIST} -- test/unit |
There was a problem hiding this comment.
this project doesn't have a tox config file
There was a problem hiding this comment.
The tox file is located here https://github.com/aws/sagemaker-tensorflow-container/blob/script-mode/tox.ini
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.