Skip to content

Commit d8393aa

Browse files
authored
Merge pull request aws#9 from aws/dependency_fix
Installation fails due to dependencies.
2 parents 662ead6 + 579b817 commit d8393aa

File tree

4 files changed

+8
-8
lines changed

4 files changed

+8
-8
lines changed

buildspec_integ.yaml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,7 @@ version: 0.2
33
phases:
44
pre_build:
55
commands:
6-
- python setup.py develop
7-
- pip install -e .[test]
6+
- pip install tox
87
build:
98
commands:
10-
- tox tests/integ/
9+
- tox tests/integ/

buildspec_unit.yaml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,7 @@ version: 0.2
33
phases:
44
pre_build:
55
commands:
6-
- python setup.py develop
7-
- pip install -e .[test]
6+
- pip install tox
87
build:
98
commands:
109
- tox tests/unit/

requirements.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,3 @@
1+
sagemaker>=1.42.8
12
boto3>=1.9.213
2-
sagemaker==1.42.0
3+
pyyaml

setup.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,9 @@ def read_version():
3030

3131
# Declare minimal set for installation
3232
required_packages = [
33+
"sagemaker>=1.42.8",
3334
"boto3>=1.9.213",
34-
"sagemaker>=1.42.8"
35+
"pyyaml"
3536
]
3637

3738
# enum is introduced in Python 3.4. Installing enum back port
@@ -67,7 +68,7 @@ def read_version():
6768
"tensorflow>=1.3.0",
6869
"mock>=2.0.0",
6970
"contextlib2==0.5.5",
70-
"IPython"
71+
"IPython",
7172
]
7273
}
7374
)

0 commit comments

Comments
 (0)