This is an automated email from the ASF dual-hosted git repository.
maximebeauchemin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git
The following commit(s) were added to refs/heads/master by this push:
new e03e276 Bump some of the requirements-dev.txt (#6700)
e03e276 is described below
commit e03e2765713919d2f91cb8ce44c57b3632390c57
Author: Maxime Beauchemin <[email protected]>
AuthorDate: Wed Jan 16 20:40:16 2019 -0800
Bump some of the requirements-dev.txt (#6700)
* Bump some of the requirements-dev.txt
* addressing comments
---
docs/_templates/layout.html | 95 ---------------------------
docs/requirements.txt | 4 +-
docs/visualization.rst | 154 ++++++++++++++++++++++----------------------
requirements-dev.txt | 12 ++--
superset/db_engine_specs.py | 4 +-
superset/db_engines/hive.py | 2 +-
superset/stats_logger.py | 3 +-
superset/viz.py | 2 +-
tests/core_tests.py | 2 +-
tox.ini | 2 +
10 files changed, 93 insertions(+), 187 deletions(-)
diff --git a/docs/_templates/layout.html b/docs/_templates/layout.html
deleted file mode 100644
index ecc205f..0000000
--- a/docs/_templates/layout.html
+++ /dev/null
@@ -1,95 +0,0 @@
-{% extends "!layout.html" %}
-
-{% set bootswatch_css_custom = ['_static/docs.css'] %}
-
-{%- block content %}
- {{ navBar() }}
- {% if pagename == 'index' %}
- <div id="carousel" class="carousel slide" data-ride="carousel">
- <!-- Indicators -->
- <ol class="carousel-indicators">
- <li data-target="#carousel" data-slide-to="0" class="active"></li>
- <li data-target="#carousel" data-slide-to="1"></li>
- <li data-target="#carousel" data-slide-to="2"></li>
- <li data-target="#carousel" data-slide-to="3"></li>
- <li data-target="#carousel" data-slide-to="4"></li>
- </ol>
-
- <!-- Wrapper for slides -->
- <div class="carousel-inner" role="listbox">
- <div class="item active text-center">
- <img src="_static/img/dash.png">
- <div class="carousel-caption">
- <div>
- <h1>Superset</h1>
- <p>
- an open source data visualization platform
- </p>
- </div>
- </div>
- </div>
- <div class="item">
- <img src="_static/img/bubble.png">
- <div class="carousel-caption">
- <h2>Explore your data
- </h2>
- <p>
- Intuitively navigate your data while slicing, dicing, and
- visualizing through a rich set of widgets
- </p>
- </div>
- </div>
- <div class="item">
- <img src="_static/img/dash.png">
- <div class="carousel-caption">
- <h2>Create and share dashboards</h2>
- <p>Assemble many data visualization "slices" into a rich
collection</p>
- </div>
- </div>
- <div class="item">
- <img src="_static/img/cloud.png">
- <div class="carousel-caption">
- <h2>Extend</h2>
- <p>Join the community and take part in extending the widget
library</p>
- </div>
- </div>
- <div class="item">
- <img src="_static/img/servers.jpg">
- <div class="carousel-caption">
- <h2>Connect</h2>
- <p>
- Access data from MySql, Presto.db, Postgres, RedShift, Oracle,
MsSql,
- SQLite, and more through the SqlAlchemy integration. You can also
- query realtime data blazingly fast out of Druid.io
- </p>
- </div>
- </div>
- </div>
- <!-- Controls -->
- <div>
- <a class="left carousel-control" href="#carousel" role="button"
data-slide="prev">
- <span class="glyphicon glyphicon-chevron-left"
aria-hidden="true"></span>
- <span class="sr-only">Previous</span>
- </a>
- <a class="right carousel-control" href="#carousel" role="button"
data-slide="next">
- <span class="glyphicon glyphicon-chevron-right"
aria-hidden="true"></span>
- <span class="sr-only">Next</span>
- </a>
- </div>
- </div>
- <hr/>
- <div class="container">
- <div class="jumbotron">
- <h1>Superset</h1>
- <p>
- is an open source data visualization platform that provides easy
- exploration of your data and allows you to create and share
- beautiful charts and dashboards
- </p>
- </div>
- </div>
- {% endif %}
- <div class="container mainbody">
- {% block body %}{% endblock %}
- </div>
-{%- endblock %}
diff --git a/docs/requirements.txt b/docs/requirements.txt
index 99f31ca..9035fea 100644
--- a/docs/requirements.txt
+++ b/docs/requirements.txt
@@ -1,2 +1,2 @@
-sphinx==1.7.1
-sphinx-rtd-theme==0.2.4
+sphinx==1.8.1
+sphinx-rtd-theme==0.3.1
diff --git a/docs/visualization.rst b/docs/visualization.rst
index f2e4636..94a787e 100644
--- a/docs/visualization.rst
+++ b/docs/visualization.rst
@@ -496,11 +496,11 @@ List of Countries
+------+------------------------------------+
|IT-PE |Pescara |
+------+------------------------------------+
-|IT-TE |Teramo |
+|IT-TE |Teramo |
+------+------------------------------------+
|IT-BA |Bari |
+------+------------------------------------+
-|IT-BT |Barletta-Andria-Trani |
+|IT-BT |Barletta-Andria-Trani |
+------+------------------------------------+
|IT-BR |Brindisi |
+------+------------------------------------+
@@ -536,177 +536,177 @@ List of Countries
+------+------------------------------------+
|IT-BO |Bologna |
+------+------------------------------------+
-|IT-FE |Ferrara |
+|IT-FE |Ferrara |
+------+------------------------------------+
|IT-FC | Forli' - Cesena |
+------+------------------------------------+
-|IT-MO |Modena |
+|IT-MO |Modena |
+------+------------------------------------+
|IT-PR |Parma |
+------+------------------------------------+
-|IT-PC |Piacenza |
+|IT-PC |Piacenza |
+------+------------------------------------+
-|IT-RA |Ravenna |
+|IT-RA |Ravenna |
+------+------------------------------------+
-|IT-RE |Reggio Nell'Emilia |
+|IT-RE |Reggio Nell'Emilia |
+------+------------------------------------+
-|IT-RN |Rimini |
+|IT-RN |Rimini |
+------+------------------------------------+
-|IT-GO |Gorizia |
+|IT-GO |Gorizia |
+------+------------------------------------+
-|IT-PN |Pordenone |
+|IT-PN |Pordenone |
+------+------------------------------------+
-|IT-TS |Trieste |
+|IT-TS |Trieste |
+------+------------------------------------+
-|IT-UD |Udine |
+|IT-UD |Udine |
+------+------------------------------------+
-|IT-FR |Frosinone |
+|IT-FR |Frosinone |
+------+------------------------------------+
-|IT-LT |Latina |
+|IT-LT |Latina |
+------+------------------------------------+
-|IT-RI |Rieti |
+|IT-RI |Rieti |
+------+------------------------------------+
-|IT-RM |Roma |
+|IT-RM |Roma |
+------+------------------------------------+
-|IT-VT |Viterbo |
+|IT-VT |Viterbo |
+------+------------------------------------+
-|IT-GE |Genova |
+|IT-GE |Genova |
+------+------------------------------------+
-|IT-IM |Imperia |
+|IT-IM |Imperia |
+------+------------------------------------+
-|IT-SP |La Spezia |
+|IT-SP |La Spezia |
+------+------------------------------------+
-|IT-SV |Savona |
+|IT-SV |Savona |
+------+------------------------------------+
-|IT-BG |Bergamo |
+|IT-BG |Bergamo |
+------+------------------------------------+
-|IT-BS |Brescia |
+|IT-BS |Brescia |
+------+------------------------------------+
-|IT-CO |Como |
+|IT-CO |Como |
+------+------------------------------------+
-|IT-CR |Cremona |
+|IT-CR |Cremona |
+------+------------------------------------+
-|IT-LC |Lecco |
+|IT-LC |Lecco |
+------+------------------------------------+
-|IT-LO |Lodi |
+|IT-LO |Lodi |
+------+------------------------------------+
-|IT-MN |Mantua |
+|IT-MN |Mantua |
+------+------------------------------------+
-|IT-MI |Milano |
+|IT-MI |Milano |
+------+------------------------------------+
-|IT-MB |Monza and Brianza |
+|IT-MB |Monza and Brianza |
+------+------------------------------------+
-|IT-PV |Pavia |
+|IT-PV |Pavia |
+------+------------------------------------+
-|IT-SO |Sondrio |
+|IT-SO |Sondrio |
+------+------------------------------------+
-|IT-VA |Varese |
+|IT-VA |Varese |
+------+------------------------------------+
-|IT-AN |Ancona |
+|IT-AN |Ancona |
+------+------------------------------------+
-|IT-AP |Ascoli Piceno |
+|IT-AP |Ascoli Piceno |
+------+------------------------------------+
-|IT-FM |Fermo |
+|IT-FM |Fermo |
+------+------------------------------------+
-|IT-MC |Macerata |
+|IT-MC |Macerata |
+------+------------------------------------+
-|IT-PU |Pesaro E Urbino |
+|IT-PU |Pesaro E Urbino |
+------+------------------------------------+
-|IT-CB |Campobasso |
+|IT-CB |Campobasso |
+------+------------------------------------+
-|IT-IS |Isernia |
+|IT-IS |Isernia |
+------+------------------------------------+
-|IT-AL |Alessandria |
+|IT-AL |Alessandria |
+------+------------------------------------+
-|IT-AT |Asti |
+|IT-AT |Asti |
+------+------------------------------------+
-|IT-BI |Biella |
+|IT-BI |Biella |
+------+------------------------------------+
-|IT-CN |Cuneo |
+|IT-CN |Cuneo |
+------+------------------------------------+
-|IT-NO |Novara |
+|IT-NO |Novara |
+------+------------------------------------+
-|IT-TO |Torino |
+|IT-TO |Torino |
+------+------------------------------------+
-|IT-VB |Verbano-Cusio-Ossola |
+|IT-VB |Verbano-Cusio-Ossola |
+------+------------------------------------+
-|IT-VC |Vercelli |
+|IT-VC |Vercelli |
+------+------------------------------------+
-|IT-CA |Cagliari |
+|IT-CA |Cagliari |
+------+------------------------------------+
-|IT-CI |Carbonia-Iglesias |
+|IT-CI |Carbonia-Iglesias |
+------+------------------------------------+
-|IT-VS |Medio Campidano |
+|IT-VS |Medio Campidano |
+------+------------------------------------+
-|IT-NU |Nuoro |
+|IT-NU |Nuoro |
+------+------------------------------------+
-|IT-OG |Ogliastra |
+|IT-OG |Ogliastra |
+------+------------------------------------+
-|IT-OT |Olbia-Tempio |
+|IT-OT |Olbia-Tempio |
+------+------------------------------------+
-|IT-OR |Oristano |
+|IT-OR |Oristano |
+------+------------------------------------+
-|IT-SS |Sassari |
+|IT-SS |Sassari |
+------+------------------------------------+
-|IT-AG |Agrigento |
+|IT-AG |Agrigento |
+------+------------------------------------+
-|IT-CL |Caltanissetta |
+|IT-CL |Caltanissetta |
+------+------------------------------------+
|IT-CT |Catania |
+------+------------------------------------+
|IT-EN |Enna |
+------+------------------------------------+
-|IT-ME |Messina |
+|IT-ME |Messina |
+------+------------------------------------+
|IT-PA |Palermo |
+------+------------------------------------+
|IT-RG |Ragusa |
+------+------------------------------------+
-|IT-SR |Syracuse |
+|IT-SR |Syracuse |
+------+------------------------------------+
|IT-TP |Trapani |
+------+------------------------------------+
-|IT-AR |Arezzo |
+|IT-AR |Arezzo |
+------+------------------------------------+
-|IT-FI |Florence |
+|IT-FI |Florence |
+------+------------------------------------+
-|IT-GR |Grosseto |
+|IT-GR |Grosseto |
+------+------------------------------------+
-|IT-LI |Livorno |
+|IT-LI |Livorno |
+------+------------------------------------+
|IT-LU |Lucca |
+------+------------------------------------+
-|IT-MS |Massa Carrara |
+|IT-MS |Massa Carrara |
+------+------------------------------------+
|IT-PI |Pisa |
+------+------------------------------------+
-|IT-PT |Pistoia |
+|IT-PT |Pistoia |
+------+------------------------------------+
|IT-PO |Prato |
+------+------------------------------------+
|IT-SI |Siena |
+------+------------------------------------+
-|IT-BZ |Bolzano |
+|IT-BZ |Bolzano |
+------+------------------------------------+
-|IT-TN |Trento |
+|IT-TN |Trento |
+------+------------------------------------+
-|IT-PG |Perugia |
+|IT-PG |Perugia |
+------+------------------------------------+
-|IT-TR |Terni |
+|IT-TR |Terni |
+------+------------------------------------+
-|IT-AO |Aosta |
+|IT-AO |Aosta |
+------+------------------------------------+
-|IT-BL |Belluno |
+|IT-BL |Belluno |
+------+------------------------------------+
-|IT-PD |Padua |
+|IT-PD |Padua |
+------+------------------------------------+
-|IT-RO |Rovigo |
+|IT-RO |Rovigo |
+------+------------------------------------+
-|IT-TV |Treviso |
+|IT-TV |Treviso |
+------+------------------------------------+
-|IT-VE |Venezia |
+|IT-VE |Venezia |
+------+------------------------------------+
-|IT-VR |Verona |
+|IT-VR |Verona |
+------+------------------------------------+
|IT-VI |Vicenza |
+------+------------------------------------+
diff --git a/requirements-dev.txt b/requirements-dev.txt
index 53dcbe7..0ac1c57 100644
--- a/requirements-dev.txt
+++ b/requirements-dev.txt
@@ -1,16 +1,16 @@
console_log==0.2.10
-python-dotenv==0.10.1
flake8-commas==2.0.0
flake8-import-order==0.18
flake8-quotes==1.0.0
-flake8==3.5.0
-flask-cors==3.0.3
+flake8==3.6.0
+flask-cors==3.0.6
ipdb==0.11
mysqlclient==1.3.13
pip-tools==3.1.0
psycopg2-binary==2.7.5
-pycodestyle==2.3.1
+pycodestyle==2.4.0
pylint==1.9.2
+python-dotenv==0.10.1
redis==2.10.6
-statsd==3.2.2
-tox==3.1.2
+statsd==3.3.0
+tox==3.5.3
diff --git a/superset/db_engine_specs.py b/superset/db_engine_specs.py
index bba5e81..1a9bf81 100644
--- a/superset/db_engine_specs.py
+++ b/superset/db_engine_specs.py
@@ -1000,7 +1000,7 @@ class HiveEngineSpec(PrestoEngineSpec):
@classmethod
def patch(cls):
- from pyhive import hive
+ from pyhive import hive # pylint: disable=no-name-in-module
from superset.db_engines import hive as patched_hive
from TCLIService import (
constants as patched_constants,
@@ -1163,7 +1163,7 @@ class HiveEngineSpec(PrestoEngineSpec):
@classmethod
def handle_cursor(cls, cursor, query, session):
"""Updates progress information"""
- from pyhive import hive
+ from pyhive import hive # pylint: disable=no-name-in-module
unfinished_states = (
hive.ttypes.TOperationState.INITIALIZED_STATE,
hive.ttypes.TOperationState.RUNNING_STATE,
diff --git a/superset/db_engines/hive.py b/superset/db_engines/hive.py
index 8bc6cb9..f0f8995 100644
--- a/superset/db_engines/hive.py
+++ b/superset/db_engines/hive.py
@@ -15,7 +15,7 @@
# specific language governing permissions and limitations
# under the License.
# pylint: disable=C,R,W
-from pyhive import hive
+from pyhive import hive # pylint: disable=no-name-in-module
from TCLIService import ttypes
from thrift import Thrift
diff --git a/superset/stats_logger.py b/superset/stats_logger.py
index bcd8459..ce4b400 100644
--- a/superset/stats_logger.py
+++ b/superset/stats_logger.py
@@ -54,8 +54,7 @@ class DummyStatsLogger(BaseStatsLogger):
def decr(self, key):
logging.debug((
- Fore.CYAN + '[stats_logger] (decr) ' + key +
- Style.RESET_ALL))
+ Fore.CYAN + '[stats_logger] (decr) ' + key + Style.RESET_ALL))
def timing(self, key, value):
logging.debug((
diff --git a/superset/viz.py b/superset/viz.py
index 7fd717b..7cf6465 100644
--- a/superset/viz.py
+++ b/superset/viz.py
@@ -2129,10 +2129,10 @@ class BaseDeckGLViz(BaseViz):
return None
try:
p = Point(s)
+ return (p.latitude, p.longitude) # pylint: disable=no-member
except Exception:
raise SpatialException(
_('Invalid spatial point encountered: %s' % s))
- return (p.latitude, p.longitude)
@staticmethod
def reverse_geohash_decode(geohash_code):
diff --git a/tests/core_tests.py b/tests/core_tests.py
index e765d93..322e974 100644
--- a/tests/core_tests.py
+++ b/tests/core_tests.py
@@ -549,7 +549,7 @@ class CoreTests(SupersetTestCase):
def test_fetch_datasource_metadata(self):
self.login(username='admin')
url = (
- '/superset/fetch_datasource_metadata?' +
+ '/superset/fetch_datasource_metadata?'
'datasourceKey=1__table'
)
resp = self.get_json_resp(url)
diff --git a/tox.ini b/tox.ini
index 6fed8b5..f23e100 100644
--- a/tox.ini
+++ b/tox.ini
@@ -20,6 +20,8 @@ ignore =
FI51
FI53
FI54
+ W504
+ W605
import-order-style = google
max-line-length = 90
require-code = true
