ci: remove travis2#581
Conversation
Makefile
Outdated
| .PHONY: unittest | ||
| unittest: ## Run unittest | ||
| python -m pytest test/unit/ | ||
| python -m pytest ${ARGS} test/unit/ |
There was a problem hiding this comment.
at least it can be $ARGS (not sure @{ARGS}), but will use $(ARGS) as well as others in this script.
azure-pipelines.yml
Outdated
| python.version: '3.8' | ||
| Python39: | ||
| python.version: '3.9' | ||
| maxParallel: 2 |
There was a problem hiding this comment.
why do we limit the amount of parallel tasks?
There was a problem hiding this comment.
Just copy-pasted one remains, so I can remove.
azure-pipelines.yml
Outdated
| # https://docs.microsoft.com/azure/devops/pipelines/languages/xcode | ||
| jobs: | ||
| - template: ./ci-jobs/functional_test.yml | ||
| - job: 'unitTests' # TODO Decide how to handle tox |
There was a problem hiding this comment.
Or we no longer need tox stuff?
I did some ways in #565 like checking out scripts from their repository or adding related things in this repository. But all of them failed.
There was a problem hiding this comment.
Or we no longer need tox stuff?
In my opinion for now, it's unnecessary for ci, but it's useful for running py37-39 in local env.
(current tox usage doesn't match with azure-pipeline since just tox includes running unit test in py37-39, as you know.)
When we look into sample for details, we might be able to use tox in azure pipeline as well.
There was a problem hiding this comment.
Probably we should tweak something more to use the template.
I tried it before, but Azure had an error 856a82f
(I forgot the exact error message, but maybe it was not able to get the template stuff)
Then, it would be nice to address only the local environment about tox in README.
https://github.com/appium/python-client#run-tests
There was a problem hiding this comment.
ok, I'll try it while this PR is on hold.
| - task: UsePythonVersion@0 | ||
| displayName: 'Use Python $(python.version)' | ||
| inputs: | ||
| versionSpec: '$(python.version)' |
There was a problem hiding this comment.
oh, nice.
When I did this matrix style, I got no matched python error. But it seems fixed on Azure side(?)
Background
from #565
Tests
https://dev.azure.com/AppiumCI/Appium%20CI/_build/results?buildId=13831&view=results
Ref
https://docs.microsoft.com/en-us/azure/devops/pipelines/ecosystems/python?view=azure-devops#run-tests-with-tox