mistercrunch closed pull request #6301: Bump some of the requirements-dev.txt
URL: https://github.com/apache/incubator-superset/pull/6301
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git a/docs/_templates/layout.html b/docs/_templates/layout.html
deleted file mode 100644
index ecc205f981..0000000000
--- 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 99f31cac65..9035feadcd 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 f2e4636214..94a787e3d4 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 53dcbe7ac7..cf47ca795f 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
-pylint==1.9.2
+pycodestyle==2.4.0
+pylint==1.9.3
+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 97b6439a36..3cb40f6c1f 100644
--- a/superset/db_engine_specs.py
+++ b/superset/db_engine_specs.py
@@ -985,7 +985,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,
@@ -1145,7 +1145,7 @@ def get_tracking_url(cls, log_lines):
@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 31c658e9da..8680e158d9 100644
--- a/superset/db_engines/hive.py
+++ b/superset/db_engines/hive.py
@@ -1,5 +1,5 @@
# 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/models/user_attributes.py
b/superset/models/user_attributes.py
index 04c56a6e0e..e987e38964 100644
--- a/superset/models/user_attributes.py
+++ b/superset/models/user_attributes.py
@@ -1,6 +1,7 @@
from flask_appbuilder import Model
-from sqlalchemy import Column, ForeignKey, Integer
-from sqlalchemy.orm import relationship
+from sqlalchemy import Column, ForeignKey, Integer # pylint:
disable=wrong-import-order
+from sqlalchemy.orm import relationship # pylint: disable=wrong-import-order
+
from superset import security_manager
from superset.models.helpers import AuditMixinNullable
diff --git a/superset/stats_logger.py b/superset/stats_logger.py
index 20f6c1ffdb..81b97f5b89 100644
--- a/superset/stats_logger.py
+++ b/superset/stats_logger.py
@@ -38,8 +38,7 @@ def incr(self, key):
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 a3c5f06b94..5f01b4cfcb 100644
--- a/superset/viz.py
+++ b/superset/viz.py
@@ -2107,10 +2107,10 @@ def parse_coordinates(s):
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 5b45cb8c54..1e6ed7613e 100644
--- a/tests/core_tests.py
+++ b/tests/core_tests.py
@@ -531,7 +531,7 @@ def test_table_metadata(self):
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 5014168c7f..f78c4a8013 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
@@ -98,7 +100,7 @@ deps =
[testenv:pylint]
commands =
- pylint superset
+ travis_wait pylint superset --verbose
deps =
-rrequirements.txt
-rrequirements-dev.txt
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
