Unify code styling (#356)

* Ignore package-lock.json
* Add a simple editorconfig
* Unified formatting and tabs to spaces
* Convert line endings to lf
* Formatting on test specs
* Clean up rollup config

Co-authored-by: ilessiivi <iless@me.com>

Author: ilessiivi

.babelrc

@@ -1,12 +1,12 @@
 {
-    "inputSourceMap": true,
-    "presets": [
-	[
-	    "@babel/preset-env",
-	    {
-		"useBuiltIns": "usage",
-		"corejs": 3,
-	    }
-	]
+  "inputSourceMap": true,
+  "presets": [
+    [
+      "@babel/preset-env",
+      {
+        "useBuiltIns": "usage",
+        "corejs": 3,
+      }
     ]
+  ]
 }

.editorconfig

@@ -0,0 +1,9 @@
+root = true
+
+[*]
+charset = utf-8
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true
+indent_style = space
+indent_size = 2

.gitignore

@@ -1,6 +1,7 @@
 *~
 
 # Node
+package-lock.json
 /build
 /node_modules
 /bower_components

CHANGELOG.md

@@ -1,52 +1,52 @@
-# 3.0.0
-
-## Breaking Changes
-
-### Events
- - Events are distincts from hooks.
- - Recognized events and hooks are now lower-snake-case (case-sensitive).
- - Hooks are "file-added", "files-added", "files-submitted". "filter-file" is a filtering hook.
- - Events are passed a native CustomEvent. IE:
-    v2: `flow.on('fileRemoved',  (file) => { ... });`
-    v3: `flow.on('file-removed', ({detail: [file]}) => { ... });`
-
-### Other
- - FlowFile does not run bootstrap() upon instanciation. This must be done manually (or rely on *addFile* functions).
-
-# 2.0.0
-
-## Features
-
- - All code follows Google javascript style guide
- - Target url can be provided with query string
- - Events **fileAdded** and **filesAdded** can prevent file from being added to $.files list by
- returning false. Custom validators can be ran here.
- - **ResumableFile.getType()** and **ResumableFile.getExtension()** helper methods added. Can be
- used for custom validation.
- - **fileProgress** and **progress** events are always asynchronous.
- - **ResumableFile.pause()** and **ResumableFile.resume()** methods for single file pausing and
- resuming.
- - **filesSubmitted** event added. Can be used to start file upload. Event is thrown then files are
- added to queue.
- - **progressCallbacksInterval** parameter added. Minimum interval between callbacks execution in
-  milliseconds.
- - **averageSpeed** and **currentSpeed** parameters added for `ResumableFile`. These params
- accuracy can be adjusted with `speedSmoothingFactor` and `progressCallbacksInterval` parameters.
- - **timeRemaining** method added for `ResumableFile`. Returns remaining time to upload in seconds. Accuracy is based on average speed.
- - **sizeUploaded** method added for `ResumableFile`. Returns size uploaded in bytes.
- - **singleFile** parameter added. Then enabled, uploaded file will replace current one.
-
-## Breaking Changes
- - **Resumable** was renamed to **Flow**
- - **ResumableFile.fileName** parameter renamed to **ResumableFile.name**
- - **Resumable.getOpt** method dropped, use Resumable.opts parameter instead if needed.
- - **Resumable.maxFiles**, **Resumable.minFileSize**, **Resumable.maxFileSize**,
- **Resumable.fileType** validators dropped. Use **fileAdded** and **filesAdded** events for
- custom validation.
- - **fileProgress** and **progress** events are not thrown on ResumableFile.abort() and ResumableFile.cancel() methods execution.
- - **cancel** event was removed. Event was always called after **Resumable.cancel()** function.
- - **fileAdded**, **filesAdded** events are thrown before file is added to upload queue. This means
- that calling **Resumable.upload()** method in these events will not start uploading current
- files. To start upload use **filesSubmitted** event instead.
- - **throttleProgressCallbacks** parameter was replaced with **progressCallbacksInterval** and it
-  is now measured in milliseconds.
+# 3.0.0
+
+## Breaking Changes
+
+### Events
+ - Events are distincts from hooks.
+ - Recognized events and hooks are now lower-snake-case (case-sensitive).
+ - Hooks are "file-added", "files-added", "files-submitted". "filter-file" is a filtering hook.
+ - Events are passed a native CustomEvent. IE:
+    v2: `flow.on('fileRemoved',  (file) => { ... });`
+    v3: `flow.on('file-removed', ({detail: [file]}) => { ... });`
+
+### Other
+ - FlowFile does not run bootstrap() upon instanciation. This must be done manually (or rely on *addFile* functions).
+
+# 2.0.0
+
+## Features
+
+ - All code follows Google javascript style guide
+ - Target url can be provided with query string
+ - Events **fileAdded** and **filesAdded** can prevent file from being added to $.files list by
+ returning false. Custom validators can be ran here.
+ - **ResumableFile.getType()** and **ResumableFile.getExtension()** helper methods added. Can be
+ used for custom validation.
+ - **fileProgress** and **progress** events are always asynchronous.
+ - **ResumableFile.pause()** and **ResumableFile.resume()** methods for single file pausing and
+ resuming.
+ - **filesSubmitted** event added. Can be used to start file upload. Event is thrown then files are
+ added to queue.
+ - **progressCallbacksInterval** parameter added. Minimum interval between callbacks execution in
+  milliseconds.
+ - **averageSpeed** and **currentSpeed** parameters added for `ResumableFile`. These params
+ accuracy can be adjusted with `speedSmoothingFactor` and `progressCallbacksInterval` parameters.
+ - **timeRemaining** method added for `ResumableFile`. Returns remaining time to upload in seconds. Accuracy is based on average speed.
+ - **sizeUploaded** method added for `ResumableFile`. Returns size uploaded in bytes.
+ - **singleFile** parameter added. Then enabled, uploaded file will replace current one.
+
+## Breaking Changes
+ - **Resumable** was renamed to **Flow**
+ - **ResumableFile.fileName** parameter renamed to **ResumableFile.name**
+ - **Resumable.getOpt** method dropped, use Resumable.opts parameter instead if needed.
+ - **Resumable.maxFiles**, **Resumable.minFileSize**, **Resumable.maxFileSize**,
+ **Resumable.fileType** validators dropped. Use **fileAdded** and **filesAdded** events for
+ custom validation.
+ - **fileProgress** and **progress** events are not thrown on ResumableFile.abort() and ResumableFile.cancel() methods execution.
+ - **cancel** event was removed. Event was always called after **Resumable.cancel()** function.
+ - **fileAdded**, **filesAdded** events are thrown before file is added to upload queue. This means
+ that calling **Resumable.upload()** method in these events will not start uploading current
+ files. To start upload use **filesSubmitted** event instead.
+ - **throttleProgressCallbacks** parameter was replaced with **progressCallbacksInterval** and it
+  is now measured in milliseconds.

babel.config.json

@@ -4,7 +4,7 @@
       "@babel/preset-env",
       {
         "useBuiltIns": "entry",
-	"corejs": "3.8"
+        "corejs": "3.8"
       }
     ]
   ]

package.js

@@ -18,7 +18,5 @@ Package.describe({
 Package.onUse(function(api) {
   api.versionsFrom(['METEOR@0.9.0', 'METEOR@1.0']); // Meteor versions
 
-  
   api.addFiles('./dist/flow.js', where); // Files in use
-
-});
+});

rollup.config.js

@@ -7,49 +7,45 @@ import resolve from '@rollup/plugin-node-resolve';
 import strip from '@rollup/plugin-strip';
 import { terser } from "rollup-plugin-terser";
 
-let plugins = [
-    resolve(),
-    commonjs(),
-    replace({
-        "<%= version %>": `'${pkg.version}'`,
-        delimiters: ["'", "'"]
-    }),
-
-    babel({ babelHelpers: 'bundled' })
+const plugins = [
+  resolve(),
+  commonjs(),
+  replace({
+    "<%= version %>": `'${pkg.version}'`,
+    delimiters: ["'", "'"]
+  }),
+  babel({
+    babelHelpers: 'bundled'
+  })
 ];
 
-export default [
-    {
-        input: 'src/Flow.js',
-        plugins,
-        output: {
-            name: 'Flow',
-            file: pkg.browser,
-            sourcemap: true,
-            format: 'umd',
-	},
-    },
-    {
-        input: 'src/Flow.js',
-        plugins: plugins.concat([strip({functions:['console.(log|debug)','debug','alert']}), terser()]),
-        output: {
-            name: 'Flow',
-            banner: `/*! ${pkg.name} ${pkg.version} */\n`,
-            file: pkg.browser.replace(/js$/, 'min.js'),
-            sourcemap: true,
-            format: 'esm',
-	},
-    },
-    {
-        input: 'src/Flow.js',
-        plugins: plugins.concat([istanbul({exclude: ['node_modules/**/*.js']})]),
-        output: {
-            name: 'Flow',
-            banner: `/*! ${pkg.name} ${pkg.version} */\n`,
-            file: pkg.browser.replace(/js$/, 'cov.js'),
-            sourcemap: true,
-            format: 'umd',
-	},
-    }
-
-];
+export default [{
+  input: 'src/Flow.js',
+  plugins,
+  output: {
+    name: 'Flow',
+    file: pkg.browser,
+    sourcemap: true,
+    format: 'umd'
+  }
+}, {
+  input: 'src/Flow.js',
+  plugins: plugins.concat(strip({ functions: ['console.(log|debug)', 'debug', 'alert'] }), terser()),
+  output: {
+    name: 'Flow',
+    banner: `/*! ${pkg.name} ${pkg.version} */\n`,
+    file: pkg.browser.replace(/js$/, 'min.js'),
+    sourcemap: true,
+    format: 'esm'
+  }
+}, {
+  input: 'src/Flow.js',
+  plugins: plugins.concat(istanbul({ exclude: ['node_modules/**/*.js'] })),
+  output: {
+    name: 'Flow',
+    banner: `/*! ${pkg.name} ${pkg.version} */\n`,
+    file: pkg.browser.replace(/js$/, 'cov.js'),
+    sourcemap: true,
+    format: 'umd'
+  }
+}];

src/Eventizer.js

@@ -219,11 +219,11 @@ export default class extends EventTarget {
     for (const [i, v] of this._events[event].entries()) {
       if ((!callback || v.listener == callback) && v.useCapture == options) {
         // console.log(`[event] Removed one callback from "${event}"`);
-	this._events[event].splice(i, 1);
+        this._events[event].splice(i, 1);
         if (! callback) {
           this._removeEventListener(event, v.listener, v);
         } else {
-	  break;
+          break;
         }
       }
     }

src/FlowChunk.js

@@ -466,7 +466,7 @@ export default class FlowChunk {
     } else {
       if (this.flowObj.opts.successStatuses.indexOf(this.xhr.status) > -1) {
         // HTTP 200, perfect
-	// HTTP 202 Accepted - The request has been accepted for processing, but the processing has not been completed.
+        // HTTP 202 Accepted - The request has been accepted for processing, but the processing has not been completed.
         return 'success';
       } else if (this.flowObj.opts.permanentErrors.indexOf(this.xhr.status) > -1 ||
                  !isTest && this.retries >= this.flowObj.opts.maxChunkRetries) {