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.

Sign up for GitHub

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

Jump to bottom

Travis ice36 #116

Merged
merged 7 commits into from
Dec 11, 2018
Merged

Travis ice36 #116

merged 7 commits into from
Dec 11, 2018

Conversation

jburel
Copy link
Member

@jburel jburel commented Dec 6, 2018

In this PR:

  • Use 16.04 image
  • Use ice 3.6
  • Change installed/upgraded version

This PR currently uses the wheel for Ubuntu from gitlab.
This will need to be modified before this PR is merged

cc @manics

@sbesson
Copy link
Member

sbesson commented Dec 6, 2018

👍 do you want to split the flake8 changes as these can probably be reviewed and merged immediately?

@jburel
Copy link
Member Author

jburel commented Dec 6, 2018

I can do but i don't think there is any rush to have that in.

manics
manics suggested changes Dec 6, 2018
View reviewed changes
Copy link
Member

@manics manics left a comment

Choose a reason for hiding this comment

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

Looks mostly OK, apart from the multi-stage DB upgrade test.

travis-build Show resolved Hide resolved
.travis.yml Outdated Show resolved Hide resolved
@manics manics merged commit 2dadbf3 into ome:master Dec 11, 2018
@jburel jburel deleted the travis-ice36 branch May 11, 2022 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@manics manics manics approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jburel @sbesson @manics