Skip to content
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

Travis CI: Add Python 3.8 production release #104

Merged
merged 2 commits into from
Nov 13, 2019

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Nov 9, 2019

Also drop the EOL Python 3.4

@stefankoegl
Copy link
Owner

Thanks! I'll have a look

Comment on lines +13 to -25
addons:
apt:
packages:
- pandoc
install:
- travis_retry pip install -r requirements.txt
- travis_retry pip install coveralls
script:
- coverage run --source=jsonpointer tests.py
after_script:
- coveralls
sudo: false
addons:
apt:
packages:
- pandoc
Copy link
Owner

Choose a reason for hiding this comment

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

Any particular reason why you moved that?, and removed the sudo: false ? I'd prefer to keep the diffs clean of unrelated changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Travis CI have completely deprecated the sudo: tag. The sudo command is always available in Travis and there is no longer anything that you can do to turn it off.

The move keeps things in chronological order because the apt tasks happen before the install tasks in the Travis CI logs.

@coveralls
Copy link

coveralls commented Nov 9, 2019

Coverage Status

Coverage increased (+0.06%) to 65.008% when pulling 9ca2e21 on cclauss:patch-1 into 911b79f on stefankoegl:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 64.914% when pulling 9ca2e21 on cclauss:patch-1 into 911b79f on stefankoegl:master.

@stefankoegl stefankoegl merged commit 3e44e04 into stefankoegl:master Nov 13, 2019
@stefankoegl
Copy link
Owner

Thanks!

@cclauss cclauss deleted the patch-1 branch November 13, 2019 19:52
@stefankoegl
Copy link
Owner

Waiting with release for stefankoegl/python-json-pointer#39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants