Skip to content

Improve testing #58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
52 changes: 44 additions & 8 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,13 +1,49 @@
sudo: false
cache:
pip: true
language: python
python:
- "2.7"
- "3.4"
- "3.5"
- "3.6"
- 2.7
- 3.4
- 3.5
- &main_python_version 3.6
before_install:
- pip install -U pip setuptools
install:
- if [[ $TRAVIS_PYTHON_VERSION == 3.5 || $TRAVIS_PYTHON_VERSION == 3.6 ]]; then pip install aiohttp==2.0.6 asynctest==0.10.0; fi
- pip install -q pep8==1.6.2
- pip install -r test-requirements-py3.txt
script:
- pep8 .
- python setup.py test
- python setup.py test

jobs:
fast_finish: true

include:
- stage: &lint_stage_name lint
python: *main_python_version
install:
- pip install pep8==1.6.2
script:
- pep8 .

- stage: &deploy_stage_name deploy to PYPI (triggered only for tagged commits)
python: *main_python_version
before_install: []
script: skip
deploy:
provider: pypi
distribution: sdist bdist_wheel
user: dblack
password:
secure: YOUR_ENCRYPTED_PASSWORD_HERE_https://docs.travis-ci.com/user/encryption-keys/
# Since commit https://github.com/travis-ci/dpl/commit/90b5e39
# it is default that Travis PYPI provider has `skip_upload_docs: true`
# set by default.
on:
tags: true
all_branches: true
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well probably only off master.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Usually you only tag master.

P.S. You can push to this branch

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want me to remove it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dbaxa ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll review this pull request again when I get a chance.


stages:
- *lint_stage_name
- test
- name: *deploy_stage_name
if: tag IS present
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@


setup(
setup_requires=['pbr==1.8.1'],
setup_requires=['pbr==3.1.1'],
pbr=True,
test_suite='nose.collector',
platforms=['any'],
Expand Down
2 changes: 2 additions & 0 deletions test-requirements-py3.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
aiohttp>=2.0,<2.3; python_version>="3.4"
asynctest==0.10.0; python_version>="3.4"