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

bump sqlalchemy and sqlalchemy-utils; update .travis.yml #211

Merged
merged 5 commits into from
Apr 22, 2019

Conversation

vpsx
Copy link
Contributor

@vpsx vpsx commented Apr 22, 2019

tldr: bump sqlalchemy and sqlalchemy-utils. Also specify jdk 8 in .travis.yml.

Tried to just bump sqlalchemy and sqlalchemy-utils. Travis error:

Package oracle-java8-set-default is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source
However the following packages replace it:
  oracle-java9-set-default

Probably because: https://launchpad.net/~webupd8team/+archive/ubuntu/java

The Travis dist was trusty (just because that is the default)(...until tomorrow!). Switching from java8 to java9 worked, as did doing

before_install:
  - jdk_switcher use oraclejdk8

in trusty. But we prefer xenial(and it's the new default), and we don't really care about jdk 8 or 9 so long as Swagger works, so settled on specifying

dist: xenial
jdk: 
    - oraclejdk8

Dependency updates

bump sqlalchemy from 1.0.8 to 1.3.3
bump sqlalchemy-utils from 0.32.21 to 0.33.11

@vpsx vpsx force-pushed the chore/sqlalchemy branch from ddfd752 to 2886579 Compare April 22, 2019 19:51
@vpsx vpsx requested a review from fantix April 22, 2019 20:13
@vpsx vpsx changed the title bump sqlalchemy and sqlalchemy-utils bump sqlalchemy and sqlalchemy-utils; update .travis.yml Apr 22, 2019
@vpsx vpsx merged commit 4666f58 into master Apr 22, 2019
@vpsx vpsx deleted the chore/sqlalchemy branch April 22, 2019 20:36
@fantix
Copy link
Contributor

fantix commented Apr 25, 2019

Sorry to comment on a closed PR: you probably need to update the versions in setup.py too.

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.

2 participants