Fix update_package_data

Author: Frédéric Collonval

.circleci/config.yml

@@ -3,7 +3,7 @@ version: 2
 jobs:
   check-code-formatting:
     docker:
-      - image: circleci/python:3.7-buster-node-browsers
+      - image: circleci/python:3.7-stretch-node-browsers
 
     steps:
       - checkout
@@ -17,10 +17,9 @@ jobs:
   # Core
   python-2.7-core:
     docker:
-      - image: circleci/python:2.7-buster-node-browsers
+      - image: circleci/python:2.7-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_27: python2.7
-    resource_class: large
 
     steps:
       - checkout
@@ -36,10 +35,9 @@ jobs:
 
   python-3.5-core:
     docker:
-      - image: circleci/python:3.5-buster-node-browsers
+      - image: circleci/python:3.5-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_35: python3.5
-    resource_class: large
 
     steps:
       - checkout
@@ -55,10 +53,9 @@ jobs:
 
   python-3.6-core:
     docker:
-      - image: circleci/python:3.6-buster-node-browsers
+      - image: circleci/python:3.6-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_36: python3.6
-    resource_class: large
 
     steps:
       - checkout
@@ -74,10 +71,9 @@ jobs:
 
   python-3.7-core:
     docker:
-      - image: circleci/python:3.7-buster-node-browsers
+      - image: circleci/python:3.7-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_37: python3.7
-    resource_class: large
 
     steps:
       - checkout
@@ -93,11 +89,10 @@ jobs:
 
   python-3.7-percy:
     docker:
-      - image: circleci/python:3.7-buster-node-browsers
+      - image: circleci/python:3.7-stretch-node-browsers
         environment:
           PERCY_ENABLED: True
           PERCY_PROJECT: plotly/plotly.py
-    resource_class: large
 
     steps:
       - checkout
@@ -127,10 +122,9 @@ jobs:
   # Optional
   python-2.7-optional:
     docker:
-      - image: circleci/python:2.7-buster-node-browsers
+      - image: circleci/python:2.7-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_27: python2.7
-    resource_class: large
 
     steps:
       - checkout
@@ -146,10 +140,9 @@ jobs:
 
   python-3.5-optional:
     docker:
-      - image: circleci/python:3.5-buster-node-browsers
+      - image: circleci/python:3.5-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_35: python3.5
-    resource_class: large
 
     steps:
       - checkout
@@ -165,10 +158,9 @@ jobs:
 
   python-3.6-optional:
     docker:
-      - image: circleci/python:3.6-buster-node-browsers
+      - image: circleci/python:3.6-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_36: python3.6
-    resource_class: large
 
     steps:
       - checkout
@@ -184,10 +176,9 @@ jobs:
 
   python-3.7-optional:
     docker:
-      - image: circleci/python:3.7-buster-node-browsers
+      - image: circleci/python:3.7-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_37: python3.7
-    resource_class: large
 
     steps:
       - checkout
@@ -204,10 +195,9 @@ jobs:
   # Plot.ly
   python-2.7-plot_ly:
     docker:
-      - image: circleci/python:2.7-buster-node-browsers
+      - image: circleci/python:2.7-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_27: python2.7
-    resource_class: large
 
     steps:
       - checkout
@@ -223,10 +213,9 @@ jobs:
 
   python-3.5-plot_ly:
     docker:
-      - image: circleci/python:3.5-buster-node-browsers
+      - image: circleci/python:3.5-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_35: python3.5
-    resource_class: large
 
     steps:
       - checkout
@@ -242,10 +231,9 @@ jobs:
 
   python-3.7-plot_ly:
     docker:
-      - image: circleci/python:3.7-buster-node-browsers
+      - image: circleci/python:3.7-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_37: python3.7
-    resource_class: large
 
     steps:
       - checkout
@@ -352,7 +340,7 @@ jobs:
 
   plotlyjs_dev_build:
     docker:
-      - image: circleci/python:3.7-buster-node-browsers
+      - image: circleci/python:3.7-stretch-node-browsers
         environment:
           PLOTLY_TOX_PYTHON_37: python3.7
           LANG: en_US.UTF-8
@@ -399,7 +387,7 @@ jobs:
     docker:
       # specify the version you desire here
       # use `-browsers` prefix for selenium tests, e.g. `3.6.1-browsers`
-      - image: circleci/python:3.7-buster-node-browsers
+      - image: circleci/python:3.7-stretch-node-browsers
 
     # working_directory: ~/project
 

packages/python/plotly/setup.py

@@ -94,23 +94,25 @@ def update_package_data(distribution):
 
     # JS assets will not be present if we are skip npm build
     if not skip_npm:
-        distribution.data_files.append(
-            (
-                "share/jupyter/labextensions/plotlywidget",
-                ["plotlywidget/labextension/package.json",],
-            ),
-            (
-                "share/jupyter/labextensions/plotlywidget/static",
-                [os.path.join(labstatic, f) for f in os.listdir(labstatic)],
-            ),
-            (
-                "share/jupyter/nbextensions/plotlywidget",
-                [
-                    "plotlywidget/nbextension/extension.js",
-                    "plotlywidget/nbextension/index.js",
-                    "plotlywidget/nbextension/index.js.LICENSE.txt",
-                ],
-            ),
+        distribution.data_files.extends(
+            [
+                (
+                    "share/jupyter/labextensions/plotlywidget",
+                    ["plotlywidget/labextension/package.json",],
+                ),
+                (
+                    "share/jupyter/labextensions/plotlywidget/static",
+                    [os.path.join(labstatic, f) for f in os.listdir(labstatic)],
+                ),
+                (
+                    "share/jupyter/nbextensions/plotlywidget",
+                    [
+                        "plotlywidget/nbextension/extension.js",
+                        "plotlywidget/nbextension/index.js",
+                        "plotlywidget/nbextension/index.js.LICENSE.txt",
+                    ],
+                ),
+            ]
         )
 
     # re-init build_py options which load package_data