Switch dev setup to use ocular-dev-tools (aurora-opensource#312)

Author: Pessimistress

.buildkite/pipeline.yml

@@ -18,15 +18,15 @@ steps:
       queue: private-default
 
   - name: ":node:"
-    command: "node test/start"
+    command: "yarn test cover && cat coverage/lcov.info | coveralls"
     plugins:
       docker-compose#v1.5.2:
         run: streetscape
     agents:
       queue: private-default
 
   - name: ":chrome:"
-    command: "sh /etc/init.d/xvfb && yarn test-browser"
+    command: "sh /etc/init.d/xvfb && yarn test browser"
     plugins:
       docker-compose#v1.5.2:
         run: streetscape

.nycrc

@@ -1,12 +1,3 @@
 {
-  "all": true,
-  "sourceMap": false,
-  "instrument": false,
-  "include": [
-    "modules/core/src/index.js",
-    "modules/core/src/**/*.js"
-  ],
-  "exclude": [
-    "test/**/*.js"
-  ]
-}
+  "extends": "node_modules/ocular-dev-tools/templates/.nycrc"
+}

CHANGELOG.md

@@ -17,7 +17,7 @@ All notable changes to XVIZ will be documented in this file. This project adhere
 
 - Upgrade to deck.gl 6.4.10 (#300)
 - Add URL passing to the XVIZ Loaders (#285)
-- Add backing missing error handler _onWSError (#295)
+- Add backing missing error handler \_onWSError (#295)
 - flip signs (#290)
 
 ## [1.0.0-beta.5]

Dockerfile

@@ -12,7 +12,7 @@ ENV PATH /streetscape/node_modules/.bin:$PATH
 ENV DISPLAY :99
 
 RUN apt-get update
-RUN apt-get -y install libxi-dev libgl1-mesa-dev xvfb
+RUN apt-get -y install libxi-dev libgl1-mesa-dev xvfb jq
 
 ADD .buildkite/xvfb /etc/init.d/xvfb
 RUN chmod a+x /etc/init.d/xvfb

babel.config.js

@@ -1,87 +1,26 @@
 // Copyright (c) 2019 Uber Technologies, Inc.
 //
-// Permission is hereby granted, free of charge, to any person obtaining a copy
-// of this software and associated documentation files (the "Software"), to deal
-// in the Software without restriction, including without limitation the rights
-// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-// copies of the Software, and to permit persons to whom the Software is
-// furnished to do so, subject to the following conditions:
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
 //
-// The above copyright notice and this permission notice shall be included in
-// all copies or substantial portions of the Software.
+//     http://www.apache.org/licenses/LICENSE-2.0
 //
-// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-// THE SOFTWARE.
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+const getBabelConfig = require('ocular-dev-tools/config/babel.config');
 
-const TARGETS = {
-  "chrome": "60",
-  "edge": "15",
-  "firefox": "53",
-  "ios": "10.3",
-  "safari": "10.1",
-  "node": "8"
-};
-
-const CONFIG = {
-  default: {
-    "presets": [
-      [ "@babel/env", {
-        "targets": TARGETS
-      }],
-      "@babel/react"
-    ],
-    "plugins": [
-      "version-inline",
-      "@babel/proposal-class-properties"
-    ],
-  }
-};
-
-CONFIG.es6 = Object.assign({}, CONFIG.default, {
-  "presets": [
-    [ "@babel/env", {
-      "targets": TARGETS,
-      "modules": false
-    }],
-    "@babel/react"
-  ]
-});
+module.exports = api => {
+  const config = getBabelConfig(api);
 
-CONFIG.esm = Object.assign({}, CONFIG.default, {
-  "presets": [
-    [ "@babel/env", {
-      "modules": false
-    }],
-    "@babel/react"
-  ]
-});
+  config.presets.push('@babel/react');
 
-CONFIG.es5 = Object.assign({}, CONFIG.default, {
-  "presets": [
-    [ "@babel/env", {
-      "modules": "commonjs"
-    }],
-    "@babel/react"
-  ],
-});
+  config.plugins = config.plugins || [];
 
-CONFIG.cover = Object.assign({}, CONFIG.default);
-CONFIG.cover.plugins = CONFIG.cover.plugins.concat(['istanbul']);
+  config.plugins.push('version-inline', '@babel/proposal-class-properties');
 
-module.exports = function getConfig(api) {
-
-  // eslint-disable-next-line
-  var env = api.cache(() => process.env.BABEL_ENV || process.env.NODE_ENV);
-
-  const config = CONFIG[env] || CONFIG.default;
-  // Uncomment to debug
-  console.error(env, config.plugins);
   return config;
 };
-
-module.exports.config = CONFIG.es6;

docs/api-reference/log-viewer.md

@@ -179,7 +179,8 @@ Callback when the canvas is right clicked on. Will receive the following argumen
 
 Callback when deck.gl is initialized. Will receive the following arguments:
 
-- `deck` (Object) - a deck.gl [Deck](http://deck.gl/#/documentation/deckgl-api-reference/deck) instance.
+- `deck` (Object) - a deck.gl [Deck](http://deck.gl/#/documentation/deckgl-api-reference/deck)
+  instance.
 
 ##### onMapLoad (Function, optional)
 

modules/core/README.md

@@ -1,5 +1,6 @@
 # @streetscape.gl/core
 
-The loaders and core rendering components of streetscape.gl, a visualizaton toolkit for 3D maps and autonomous XVIZ protocol data.
+The loaders and core rendering components of streetscape.gl, a visualizaton toolkit for 3D maps and
+autonomous XVIZ protocol data.
 
 [Home](https://avs.auto)

modules/core/package.json

@@ -15,11 +15,6 @@
     "README.md"
   ],
   "scripts": {
-    "clean": "rm -fr dist && mkdir -p dist",
-    "build-es6": "BABEL_ENV=es6 babel src --config-file ../../babel.config.js --out-dir dist/es6 --source-maps --ignore 'node_modules/'",
-    "build-esm": "BABEL_ENV=esm babel src --config-file ../../babel.config.js --out-dir dist/esm --source-maps --ignore 'node_modules/'",
-    "build-es5": "BABEL_ENV=es5 babel src --config-file ../../babel.config.js --out-dir dist/es5 --source-maps --ignore 'node_modules/'",
-    "build": "npm run clean && npm run build-es6 && npm run build-esm && npm run build-es5"
   },
   "dependencies": {
     "@deck.gl/core": "^7.0.4",

modules/core/src/loaders/xviz-websocket-loader.js

@@ -167,5 +167,4 @@ export default class XVIZWebsocketLoader extends XVIZLoaderInterface {
   _onWSError = event => {
     this._debug('socket_error', event);
   };
-
 }

modules/layers/README.md

@@ -1,5 +1,6 @@
 # @streetscape.gl/layers
 
-Additional [deck.gl](http://deck.gl) map layers for streetscape.gl, a visualizaton toolkit for 3D maps and autonomous XVIZ protocol data.
+Additional [deck.gl](http://deck.gl) map layers for streetscape.gl, a visualizaton toolkit for 3D
+maps and autonomous XVIZ protocol data.
 
 [Home](https://avs.auto)