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

[ENH] Automatically create required SQL extensions #1395

Merged
merged 2 commits into from
Jul 1, 2016

Conversation

lanzagar
Copy link
Contributor

@lanzagar lanzagar commented Jul 1, 2016

Try to create the required extensions if they do not exist on the selected database.
Show error if unable to do so.

Fixes #1388

@codecov-io
Copy link

codecov-io commented Jul 1, 2016

Current coverage is 87.61%

Merging #1395 into master will not change coverage

@@             master      #1395   diff @@
==========================================
  Files            75         75          
  Lines          7460       7460          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           6536       6536          
  Misses          924        924          
  Partials          0          0          

Sunburst

Powered by Codecov. Last updated by be46877...6c93cd4

@@ -176,6 +178,7 @@ def connect(self):

def refresh_tables(self):
self.tablecombo.clear()
[self.error(42 + i) for i in range(len(EXTENSIONS))]
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not clever. 😛

@astaric astaric changed the title [ENH] Create required SQL extensions (fixes #1388) [ENH] Automatically create required SQL extensions Jul 1, 2016
@astaric astaric merged commit ecc474d into biolab:master Jul 1, 2016
@lanzagar lanzagar deleted the create_ext branch March 14, 2022 14:41
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.

4 participants