Migrate src->lib, add coveralls, fix mrodrig/json-2-csv-cli#4 (#172)

Migrate src directory to lib directory to follow best practices

Adds .coveralls.yml to .npmignore

Fixes: mrodrig/json-2-csv-cli#4

Author: mrodrig

.npmignore

@@ -9,3 +9,4 @@ coverage
 upgrade_guides
 tslint.json
 _config.yml
+.coveralls.yml

src/constants.json renamed to lib/constants.json

@@ -39,8 +39,10 @@ module.exports = {
 function buildOptions(opts) {
     opts = {...constants.defaultOptions, ...opts || {}};
 
-    // Note: Object.assign does a shallow default, we need to deep copy the delimiter object
-    opts.delimiter = {...constants.defaultOptions.delimiter, ...opts.delimiter};
+    // Copy the delimiter fields over since the spread operator does a shallow copy
+    opts.delimiter.field = opts.delimiter.field || constants.defaultOptions.delimiter.field;
+    opts.delimiter.wrap = opts.delimiter.wrap || constants.defaultOptions.delimiter.wrap;
+    opts.delimiter.eol = opts.delimiter.eol || constants.defaultOptions.delimiter.eol;
 
     // Otherwise, send the options back
     return opts;

src/converter.d.ts renamed to lib/converter.d.ts

@@ -5,23 +5,24 @@
     },
     "name": "json-2-csv",
     "description": "A JSON to CSV and CSV to JSON converter that natively supports sub-documents and auto-generates the CSV heading.",
-    "version": "3.7.12",
+    "version": "3.7.13",
+    "homepage": "https://mrodrig.github.io/json-2-csv",
     "repository": {
         "type": "git",
         "url": "https://github.com/mrodrig/json-2-csv.git"
     },
     "bugs": {
         "url": "https://github.com/mrodrig/json-2-csv/issues"
     },
-    "main": "./src/converter.js",
-    "types": "./src/converter.d.ts",
+    "main": "lib/converter.js",
+    "types": "./lib/converter.d.ts",
     "scripts": {
         "test": "mocha test/tests.js",
         "coverage": "nyc --reporter=lcov --reporter=text-summary --reporter=text _mocha",
         "coveralls": "nyc report --reporter=text-lcov | coveralls",
         "lint": "npm run lint:eslint && npm run lint:tslint",
-        "lint:eslint": "eslint src test",
-        "lint:tslint": "tslint -c tslint.json 'src/**/*.ts'"
+        "lint:eslint": "eslint lib test",
+        "lint:tslint": "tslint -c tslint.json 'lib/**/*.ts'"
     },
     "keywords": [
         "json",

src/converter.js renamed to lib/converter.js

@@ -3,9 +3,9 @@
 /* eslint no-unused-vars: ["error", { "varsIgnorePattern": "should" }]*/
 
 let should = require('should'),
-    converter = require('../src/converter'),
-    constants = require('../src/constants'),
-    utils = require('../src/utils'),
+    converter = require('../lib/converter'),
+    constants = require('../lib/constants'),
+    utils = require('../lib/utils'),
     defaultOptions = constants.defaultOptions;
 
 function runTests(jsonTestData, csvTestData) {