Merge pull request #1 from dailyrandomphoto/1.0.0

1.0.0

.gitignore

@@ -1,61 +1,4 @@
-# Logs
-logs
-*.log
-npm-debug.log*
-yarn-debug.log*
-yarn-error.log*
-
-# Runtime data
-pids
-*.pid
-*.seed
-*.pid.lock
-
-# Directory for instrumented libs generated by jscoverage/JSCover
-lib-cov
-
-# Coverage directory used by tools like istanbul
-coverage
-
-# nyc test coverage
-.nyc_output
-
-# Grunt intermediate storage (http://gruntjs.com/creating-plugins#storing-task-files)
-.grunt
-
-# Bower dependency directory (https://bower.io/)
-bower_components
-
-# node-waf configuration
-.lock-wscript
-
-# Compiled binary addons (https://nodejs.org/api/addons.html)
-build/Release
-
-# Dependency directories
 node_modules/
-jspm_packages/
-
-# TypeScript v1 declaration files
-typings/
-
-# Optional npm cache directory
-.npm
-
-# Optional eslint cache
-.eslintcache
-
-# Optional REPL history
-.node_repl_history
-
-# Output of 'npm pack'
-*.tgz
-
-# Yarn Integrity file
-.yarn-integrity
-
-# dotenv environment variables file
-.env
-
-# next.js build output
-.next
+package-lock.json
+yarn.lock
+.nyc_output/

.npmrc

@@ -0,0 +1 @@
+package-lock=false

.travis.yml

@@ -1,19 +1,13 @@
 language: node_js
 
-sudo: false
-
-cache:
-  apt: true
-  directories:
-    - node_modules
-
 node_js:
   - "8"
   - "10"
-  - "node"
+  - "12"
+  - "13"
 
 script:
-  - npm run eslint
+  - npm run lint
   - npm run test-cov
 
 after_script:

README.md

@@ -1,7 +1,6 @@
 # node-serialization
 
 [![NPM Version][npm-version-image]][npm-url]
-[![NPM Downloads][npm-downloads-image]][npm-url]
 [![LICENSE][license-image]][license-url]
 [![Build Status][travis-image]][travis-url]
 [![Coverage Status][coveralls-image]][coveralls-url]
@@ -110,10 +109,10 @@ Copyright (c) 2019 dailyrandomphoto. Licensed under the [MIT license][license-ur
 [dependencies-url]: https://david-dm.org/dailyrandomphoto/node-serialization
 [devDependencies-url]: https://david-dm.org/dailyrandomphoto/node-serialization?type=dev
 
-[npm-downloads-image]: https://img.shields.io/npm/dm/node-serialization.svg
-[npm-version-image]: https://img.shields.io/npm/v/node-serialization.svg
-[license-image]: https://img.shields.io/npm/l/node-serialization.svg
-[travis-image]: https://img.shields.io/travis/dailyrandomphoto/node-serialization/master
-[coveralls-image]: https://coveralls.io/repos/github/dailyrandomphoto/node-serialization/badge.svg?branch=master
-[dependencies-image]: https://david-dm.org/dailyrandomphoto/node-serialization/status.svg
-[devDependencies-image]: https://david-dm.org/dailyrandomphoto/node-serialization/dev-status.svg
+[npm-downloads-image]: https://img.shields.io/npm/dm/node-serialization
+[npm-version-image]: https://img.shields.io/npm/v/node-serialization
+[license-image]: https://img.shields.io/npm/l/node-serialization
+[travis-image]: https://img.shields.io/travis/dailyrandomphoto/node-serialization
+[coveralls-image]: https://img.shields.io/coveralls/github/dailyrandomphoto/node-serialization
+[dependencies-image]: https://img.shields.io/david/dailyrandomphoto/node-serialization
+[devDependencies-image]: https://img.shields.io/david/dev/dailyrandomphoto/node-serialization

lib/convert.js

@@ -1,17 +1,17 @@
 const serialization = require('./serialization');
 
 function convert(srcFile, srcDeserialize, destFile, destSerialize) {
-  const { readFile } = serialization(null, srcDeserialize);
-  const { writeFile } = serialization(destSerialize, null);
+  const {readFile} = serialization(null, srcDeserialize);
+  const {writeFile} = serialization(destSerialize, null);
 
-  return readFile(srcFile).then((content) => writeFile(destFile, content));
+  return readFile(srcFile).then(content => writeFile(destFile, content));
 }
 
 function convertSync(srcFile, srcDeserialize, destFile, destSerialize) {
-  const { readFileSync } = serialization(null, srcDeserialize);
-  const { writeFileSync } = serialization(destSerialize, null);
+  const {readFileSync} = serialization(null, srcDeserialize);
+  const {writeFileSync} = serialization(destSerialize, null);
 
   writeFileSync(destFile, readFileSync(srcFile));
 }
 
-module.exports = { convert, convertSync };
+module.exports = {convert, convertSync};

lib/index.js

@@ -1,20 +1,20 @@
-const { serialize, deserialize } = require('v8');
+const {serialize, deserialize} = require('v8');
 const serialization = require('./serialization');
 const {
   serializeJson,
   deserializeJson,
   readJson,
   writeJson,
   readJsonSync,
-  writeJsonSync,
+  writeJsonSync
 } = require('./json');
-const { convert, convertSync } = require('./convert');
+const {convert, convertSync} = require('./convert');
 
 const {
   readFile,
   writeFile,
   readFileSync,
-  writeFileSync,
+  writeFileSync
 } = serialization(serialize, deserialize);
 
 module.exports = {
@@ -35,5 +35,5 @@ module.exports = {
   writeJsonSync,
 
   convert,
-  convertSync,
+  convertSync
 };

lib/json.js

@@ -1,20 +1,22 @@
 const fs = require('fs-extra');
 
-const serializeJson = (obj) => JSON.stringify(obj);
+const serializeJson = obj => JSON.stringify(obj);
 
-const deserializeJson = (str) => JSON.parse(str);
+const deserializeJson = str => JSON.parse(str);
 
 const writeJson = (file, contents) => {
   let file2 = file;
   if (typeof file === 'string') {
     file2 = `${file}.writing`;
   }
+
   return fs.writeJson(file2, contents)
     .then(() => {
       if (file2 !== file) {
-        fs.rename(file2, file, (err) => {
-          if (err) throw err;
-        });
+        return fs.move(file2, file, {overwrite: true})
+          .catch(error => {
+            throw error;
+          });
       }
     });
 };
@@ -24,6 +26,7 @@ const writeJsonSync = (file, contents) => {
   if (typeof file === 'string') {
     file2 = `${file}.writing`;
   }
+
   fs.writeJsonSync(file2, contents);
   if (file2 !== file) {
     fs.renameSync(file2, file);
@@ -36,5 +39,5 @@ module.exports = {
   readJson: fs.readJson,
   writeJson,
   readJsonSync: fs.readJsonSync,
-  writeJsonSync,
+  writeJsonSync
 };

lib/serialization.js

@@ -1,22 +1,23 @@
-// const { serialize, deserialize } = require('v8');
 const fs = require('fs-extra');
 
 function serialization(serialize, deserialize) {
-  const readFile = (file) => fs.readFile(file).then((data) => deserialize(data));
+  const readFile = file => fs.readFile(file).then(data => deserialize(data));
 
-  const readFileSync = (file) => deserialize(fs.readFileSync(file));
+  const readFileSync = file => deserialize(fs.readFileSync(file));
 
   const writeFile = (file, contents) => {
     let file2 = file;
     if (typeof file === 'string') {
       file2 = `${file}.writing`;
     }
+
     return fs.writeFile(file2, serialize(contents))
       .then(() => {
         if (file2 !== file) {
-          fs.rename(file2, file, (err) => {
-            if (err) throw err;
-          });
+          return fs.move(file2, file, {overwrite: true})
+            .catch(error => {
+              throw error;
+            });
         }
       });
   };
@@ -26,6 +27,7 @@ function serialization(serialize, deserialize) {
     if (typeof file === 'string') {
       file2 = `${file}.writing`;
     }
+
     fs.writeFileSync(file2, serialize(contents));
     if (file2 !== file) {
       fs.renameSync(file2, file);
@@ -38,9 +40,8 @@ function serialization(serialize, deserialize) {
     readFile,
     writeFile,
     readFileSync,
-    writeFileSync,
+    writeFileSync
   };
 }
 
-
 module.exports = serialization;