-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Conversation
- script: | | ||
python3 -V | ||
python3 -m pip install --upgrade pip setuptools --user | ||
python3 -m pip install --upgrade nni --user |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This only tests released version right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So when this test case fails, we have already published a broken release...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just a start point, we can add more later, for example:
- pre-release testpypi test
- make dev install
- install for different platforms (python version, virtual envs)
- anything related installation not covered by IT.
test/pipelines-it-installation.yml
Outdated
steps: | ||
- script: | | ||
python -V | ||
python3 -m pip install --upgrade pip setuptools --user |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why python3
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, fixed.
Only test pip install for now, will add more installation test later.
This test is scheduled daily.