Merge branch 'main' into worklets-instance-mode

Author: montoyamoraga

.eslintignore

@@ -1,4 +1,3 @@
 Gruntfile.js
-test/test.js
 webpack.config.js
 lib/

.eslintrc

@@ -13,67 +13,29 @@
     "Float32Array": true,
     "Uint8Array": true
   },
+  "extends": ["eslint:recommended", "prettier"],
+  "plugins": ["prettier"],
   "rules": {
-    "block-scoped-var": 0,
-    "camelcase": 0,
-    "comma-style": [
-      2,
-      "last"
-    ],
-    "dot-notation": [
+    "prettier/prettier": ["error"],
+    "no-cond-assign": [2, "except-parens"],
+    "eqeqeq": ["error", "smart"],
+    "no-use-before-define": [
       2,
       {
-        "allowKeywords": true
+        "functions": false
       }
     ],
-    "eqeqeq": [
-      2,
-      "allow-null"
-    ],
-    "eol-last": ["error", "always"],
-    "guard-for-in": 2,
-    "indent": ["error", 2, { "SwitchCase": 1 }],
-    "max-len": [1, 120, 2, { "ignoreComments": true }],
-    "new-cap": 2,
+    "new-cap": 0,
     "no-caller": 2,
-    "no-cond-assign": [
-      2,
-      "except-parens"
-    ],
-    "no-debugger": 2,
-    "no-empty": 2,
-    "no-eval": 2,
-    "no-extend-native": 2,
-    "no-extra-parens": 2,
-    "no-extra-semi": 2,
-    "no-irregular-whitespace": 2,
-    "no-trailing-spaces": 2,
-    "no-iterator": 2,
-    "no-loop-func": 0,
-    "no-multiple-empty-lines": ["error", { "max": 2, "maxEOF": 1 }],
-    "no-multi-str": 2,
-    "no-new": 2,
-    "no-plusplus": 0,
-    "no-proto": 2,
-    "no-script-url": 2,
-    "no-sequences": 2,
-    "no-shadow": 2,
-    "no-undef": 2,
-    "no-unused-vars": 2,
-    "no-with": 2,
-    "quotes": [
-      2,
-      "single"
-    ],
-    "semi": [
-      2
-    ],
-    "space-before-blocks": "error",
-    "strict": 2,
-    "valid-typeof": 2,
-    "wrap-iife": [
-      2,
-      "inside"
-    ]
-  }
+    "no-undef": 0,
+    "no-unused-vars": ["error", { "args": "none" }],
+    "no-empty": ["error", { "allowEmptyCatch": true }],
+    "no-console": "off"
+   
+  },
+  "parserOptions": {
+    "ecmaVersion": 8,
+    "sourceType": "module",
+    "allowImportExportEverywhere": true
+    }
 }

.github/FUNDING.yml

@@ -0,0 +1,2 @@
+github: processing
+custom: https://processingfoundation.org/

.github/workflows/ci-lint.yml

@@ -0,0 +1,28 @@
+# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node
+# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
+
+name: Linting
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  lint:
+
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [10.x, 12.x]
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Use Node.js ${{ matrix.node-version }}
+      uses: actions/setup-node@v1
+      with:
+        node-version: ${{ matrix.node-version }}
+    - run: npm ci
+    - run: npm run lint

.github/workflows/ci-test.yml

@@ -0,0 +1,30 @@
+name: Testing
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        node-version: [10.x, 12.x]
+
+
+    steps:
+      - uses: actions/checkout@v1
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v1
+        with:
+          node-version: ${{ matrix.node-version }}
+      - name: Get node modules
+        run: npm ci
+        env:
+          CI: true
+      - name: Build
+        run: npm run build
+        env:
+          CI: true

.gitignore

@@ -4,3 +4,4 @@ examples/__test
 p5soundnotes
 *.DS_Store
 .vscode
+lib/p5.*

.prettierignore

@@ -0,0 +1,3 @@
+Gruntfile.js
+webpack.config.js
+lib/

.prettierrc

@@ -0,0 +1,3 @@
+{
+  "singleQuote": true
+}

.travis.yml

@@ -4,11 +4,36 @@ module.exports = function(grunt) {
 
   grunt.initConfig({
     pkg: grunt.file.readJSON('package.json'),
-
+    decomment: {
+      any: {
+        // remove comments added by webpack from the build
+        files: {
+            "./lib/p5.sound.js": "./lib/p5.sound.js",
+        },
+        options: {
+          ignore: [
+            // keep JSDoc comments (p5.js repo's YUIDoc task parses those for documentation)
+            /\/\*\*\s*\n([^\*]|(\*(?!\/)))*\*\//g,
+            // keep the version number
+            /.*Version.*/
+          ]
+        }
+      }
+    },
     // Configure style consistency
     eslint: {
       source: {
-        options: {configFile: './.eslintrc'},
+        options: {
+          configFile: './.eslintrc',
+          fix: true
+        },
+        src: ['src/**/*.js', 'test/tests/**/*.js']
+      },
+      sourceNofix: {
+        options: {
+          configFile: './.eslintrc',
+          fix: false
+        },
         src: ['src/**/*.js', 'test/tests/**/*.js']
       }
     },
@@ -38,18 +63,29 @@ module.exports = function(grunt) {
           hostname: '*'
         }
       }
-    }
+    },
+    githooks: {
+      all: {
+        options:{
+          template:"templates/pre-commit-hook.js"
+        },
+      'pre-commit':'lint-nofix' //runs elint in -nofix mode  before every git commit 
+      }
+      }
   });
 
-
+  
   grunt.loadNpmTasks('grunt-webpack');
   grunt.loadNpmTasks('grunt-eslint');
   grunt.loadNpmTasks('grunt-contrib-connect');
   grunt.loadNpmTasks('grunt-open');
+  grunt.loadNpmTasks('grunt-decomment');
+  grunt.loadNpmTasks('grunt-githooks');
 
   grunt.registerTask('lint', ['eslint:source']);
-  grunt.registerTask('default', ['webpack:prod']);
-  grunt.registerTask('dev', ['connect','webpack:dev']);
+  grunt.registerTask('lint-nofix', ['eslint:sourceNofix']);
+  grunt.registerTask('default', ['webpack:prod', 'decomment']);
+  grunt.registerTask('dev', ['eslint','connect','webpack:dev', 'decomment']);
   grunt.registerTask('serve', 'connect:server:keepalive');
   grunt.registerTask('run-tests', ['serve', 'open']);
 };