rollup fixes

.eslintrc.json

@@ -0,0 +1,23 @@
+{
+  "env": {
+    "browser": true,
+    "es6": true
+  },
+  "globals": {
+    "ENV": true
+  },
+  "extends": "eslint:recommended",
+  "parserOptions": {
+    "sourceType": "module"
+  },
+  "rules": {
+    "linebreak-style": "off",
+    "comma-dangle": "off",
+    "max-len": ["error", { "code": 160 }],
+    "no-else-return": "off",
+    "eqeqeq": ["error", "always", { "null": "ignore" }],
+    "arrow-body-style": "off", 
+    "no-plusplus": "off",
+    "no-undef": "off" 
+  }
+}

.jshintrc

@@ -0,0 +1,3 @@
+{
+    "esversion": 6
+}

packages/nlmaps-leaflet/.eslintrc.json

@@ -18,7 +18,6 @@
     "eqeqeq": ["error", "always", { "null": "ignore" }],
     "arrow-body-style": "off", // ["error", "as-needed"],
     "no-plusplus": "off",
-    "no-else-return": "off",
     "no-undef": "off" // is set to off as old-school non-es6 module imports are frequently used.
   }
 }

packages/nlmaps-leaflet/config/rollup.config.js

@@ -24,12 +24,9 @@ export default config => {
         main: true,
         browser: true
       }),
-      eslint({
-        exclude: [
-          '../../../node_modules/**'
-        ],
+      eslint({        
         include: [
-          '../../lib/**'
+          '../lib/**'
         ]
       }),
       babel({

packages/nlmaps-leaflet/config/rollup.test.js

@@ -27,8 +27,8 @@ export default {
       browser: true
     }),
     eslint({
-      exclude: [
-        '../../../node_modules/**'
+      include: [
+        '../../lib/**'
       ]
     }),
     babel({

packages/nlmaps-leaflet/test/browser-test.js

@@ -41,7 +41,7 @@ test('nlmaps can create a layer object', function(t) {
   t.end();
 })
 
-test('geolocator', function(t) {
+test('geolocator', function() {
   resetMapDiv('map');
   let map = L.map('map').setView([52, 5], 10);
   let layer = bgLayer('pastel');

packages/nlmaps-openlayers/.eslintrc.json

@@ -16,9 +16,8 @@
     "max-len": ["error", { "code": 160 }],
     "no-else-return": "off",
     "eqeqeq": ["error", "always", { "null": "ignore" }],
-    "arrow-body-style": "off", // ["error", "as-needed"],
+    "arrow-body-style": "off", 
     "no-plusplus": "off",
-    "no-else-return": "off",
-    "no-undef": "off" // is set to off as old-school non-es6 module imports are frequently used.
+    "no-undef": "off" 
   }
 }

packages/nlmaps-openlayers/test/browser-test.js

@@ -11,7 +11,6 @@ export default function browserTest(){
   map.addLayer(layer);
   let geolocator = geoLocator();
   let control = bgLayer.geoLocatorControl(geolocator, map)
-  map.addControl(control)
-  console.log(control)
+  map.addControl(control)  
 
 }

packages/nlmaps/config/rollup.cjs.js

@@ -1,8 +1,10 @@
 import config from './rollup.config';
 
 export default config({
-  format: 'cjs',
-  dest: 'build/nlmaps.cjs.js',
+  output: {
+    format: 'cjs',
+    file: 'build/nlmaps.cjs.js',
+  },
   browser: false,
   external: [
     'nlmaps-googlemaps',

packages/nlmaps/config/rollup.config.js

@@ -11,12 +11,14 @@ export default config => {
   return {
     input: 'src/index.js',
     output: {
-      file: config.dest,
-      format: config.format,
-      name: config.format === 'iife' ? 'window' : 'nlmaps',
+      file: config.output.file,
+      format: config.output.format,
+      name: config.output.format === 'iife' ? 'window' : 'nlmaps',
+      extend: config.output.format === 'iife' ? true : false,
+      sourcemap:true
     },
     external: config.external,
-    extend: config.format === 'iife' ? true : false,
+
     plugins: [
       commonjs(),
       resolve({

packages/nlmaps/config/rollup.es.js

@@ -1,11 +1,14 @@
 import config from './rollup.config';
 
 export default config({
-  format: 'es',
-  dest: 'build/nlmaps.es.js',
+  output: {
+    format: 'es',
+    file: 'build/nlmaps.es.js',
+  },
   browser: false,
-  external: ['nlmaps-googlemaps',
-  'nlmaps-leaflet',
-  'nlmaps-openlayers'
+  external: [
+    'nlmaps-googlemaps',
+    'nlmaps-leaflet',
+    'nlmaps-openlayers'
   ]
 })

packages/nlmaps/config/rollup.iife.js

@@ -1,8 +1,10 @@
 import config from './rollup.config';
 
 export default config({
-  format: 'iife',
-  dest: 'build/nlmaps.iife.js',
+  output: {
+    format: 'iife',
+    file: 'build/nlmaps.iife.js',
+  },
   browser: true,
   external: ''
 })

packages/nlmaps/config/rollup.test.js

@@ -9,10 +9,12 @@ import builtins from 'rollup-plugin-node-builtins';
 import globals from 'rollup-plugin-node-globals';
 
 export default {
-  entry: 'test/browser-test.js',
-  format: 'iife',
-  moduleName: 'testing',
-  dest: 'build/browser-test.js',
+  input: 'test/browser-test.js',
+  output: { 
+    format: 'iife',
+    name: 'testing',
+    file: 'build/browser-test.js',
+  },
   plugins: [
     commonjs(),
     globals(),