Skip to content
New issue

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

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

Already on GitHub? # to your account

travis: Run tests on 16.04 for Python 3.7, 3.8-dev #124

Merged
merged 1 commit into from
Dec 7, 2018

Conversation

kba
Copy link
Contributor

@kba kba commented Dec 7, 2018

With this change, tests on Travis will run on 16.04 containers instead of 14.04 (for which 3.7 and 3.8-dev seem unavailable).

c.f. https://travis-ci.org/kba/bagit-python/builds/464906319

BTW The coveralls setup appears broken, the configuration file has been removed from the repo and there is an error Travis:

Submitting coverage to coveralls.io...
Coverage submitted!
Couldn't find a repository matching this job.
Traceback (most recent call last):
  File "/home/travis/virtualenv/python3.7.1/bin/coveralls", line 11, in <module>
    sys.exit(main())
  File "/home/travis/virtualenv/python3.7.1/lib/python3.7/site-packages/coveralls/cli.py", line 80, in main
    log.info(result['url'])
KeyError: 'url'

@acdha
Copy link
Member

acdha commented Dec 7, 2018

Coveralls doesn't need the config file. They must have changed something on their side; I'll see if they have any useful diagnostic information.

@acdha acdha merged commit 8a8263e into LibraryOfCongress:master Dec 7, 2018
@acdha
Copy link
Member

acdha commented Dec 7, 2018

Oh, that's because it's your build rather than ours — I'm assuming you could register your repo at coveralls.io but otherwise it seems to be working.

@kba kba deleted the travis-on-xenial branch December 7, 2018 15:39
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 83.502% when pulling fb6c7b1 on kba:travis-on-xenial into c39b650 on LibraryOfCongress:master.

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

Successfully merging this pull request may close these issues.

3 participants