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

Remove google compute engine #439

Merged
merged 1 commit into from
Jun 18, 2019

Conversation

PrimozGodec
Copy link
Collaborator

@PrimozGodec PrimozGodec commented Jun 18, 2019

Issue

Google compute engine was added as a requirement because Travis test was failing because of issues with boto. The issue is still present but since google-compute-engine is not available on Windows it must be installed only on Linux and Mac OS.

Description of changes

Google compute engine is limited to Linux and Mac OS.

Includes
  • Code changes
  • Tests
  • Documentation

@PrimozGodec PrimozGodec force-pushed the remove-google-compute-engine branch from 821dd36 to e1c9275 Compare June 18, 2019 11:41
@PrimozGodec PrimozGodec force-pushed the remove-google-compute-engine branch from e1c9275 to 98ec3e7 Compare June 18, 2019 11:54
@codecov-io
Copy link

Codecov Report

Merging #439 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #439   +/-   ##
=======================================
  Coverage   86.06%   86.06%           
=======================================
  Files          34       34           
  Lines        1931     1931           
  Branches      331      331           
=======================================
  Hits         1662     1662           
  Misses        229      229           
  Partials       40       40

@PrimozGodec PrimozGodec merged commit b0cc092 into biolab:master Jun 18, 2019
@PrimozGodec PrimozGodec deleted the remove-google-compute-engine branch June 18, 2019 12:05
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