Use latest Travis-CI image.#109
Conversation
|
@lsetiawan can you take a quick look? I removed the Travis-CI image pin b/c that image will be removed soon. I got all the tests to work as expected by removing the unnecessary stuff in the |
| - postgresql | ||
|
|
||
| addons: | ||
| postgresql: "9.5" |
There was a problem hiding this comment.
I assume mysql is built in already?
There was a problem hiding this comment.
postgresql too but somehow it will be 2.3 😕
So I guess we need to specify a newer version.
There was a problem hiding this comment.
2.3 is the postgis version. I think explicit definition is better so that we know exactly what database versions and extension versions we are using. 😃
There was a problem hiding this comment.
No idea how to specify version of mysql though.
There was a problem hiding this comment.
I tried a few things and they all failed. I don't really like to pin versions unless we don't have a choice. Without pinning we can catch errors early when something is updated.
lsetiawan
left a comment
lsetiawan
left a comment
There was a problem hiding this comment.
All looks good. Merging now. 😄
I generally agree with this approach regarding software libraries, but in the case of database systems or other such heavy applications (eg, THREDDS), I prefer to pin to specific versions, as @lsetiawan suggested. |
Travis-CI uses Ubuntu Trusty. @lsetiawan do you use Ubuntu? Can you suggest a package name-version for me to try in the |
I do use ubuntu. What ubuntu version does travis use? |
|
They say the latest image is |
|
Will do in a moment. (Switched to Windows to solved something very quickly.) |
|
No rush 😄 |
3 participants
Closes #88