Migrate to github actions, upgrade min es version and remove bower

sagiegurari · sagiegurari · commit c166cf617c44 · 2020-05-11T21:51:31.000Z
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -1,5 +1,5 @@
 'use strict';
 
-var commons = require('js-project-commons');
+const commons = require('js-project-commons');
 
 module.exports = commons.lint.eslint.web;
diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md
@@ -12,7 +12,7 @@ Fixed a bug or just want to provided additional functionality?<br>
 Simply fork this repository, implement your changes and create a pull request.<br>
 Few guidelines regarding pull requests:
 
-* This repository is integrated with travis.ci for continuous integration.<br>
+* This repository is integrated with github actions for continuous integration.<br>
 
 Your pull request build must pass (the build will run automatically).<br>
 You can run the following command locally to ensure the build will pass:
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -0,0 +1,26 @@
+name: CI
+on: [push, pull_request]
+env:
+  CLICOLOR_FORCE: 1
+jobs:
+  ci:
+    name: CI
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v2
+    - name: Install
+      uses: ianwalter/puppeteer@v2.0.0
+      with:
+        args: npm install
+    - name: Run CI
+      uses: ianwalter/puppeteer@v2.0.0
+      with:
+        args: npm test
+    - name: Coveralls
+      uses: coverallsapp/github-action@master
+      with:
+        github-token: ${{ secrets.GITHUB_TOKEN }}
+        path-to-lcov: './target/coverage/report/lcov.info'
diff --git a/.travis.yml b/.travis.yml
diff --git a/README.md b/README.md
@@ -1,6 +1,6 @@
 # simple-web-notification
 
-[![NPM Version](http://img.shields.io/npm/v/simple-web-notification.svg?style=flat)](https://www.npmjs.org/package/simple-web-notification) [![Build Status](https://travis-ci.org/sagiegurari/simple-web-notification.svg)](http://travis-ci.org/sagiegurari/simple-web-notification) [![Coverage Status](https://coveralls.io/repos/sagiegurari/simple-web-notification/badge.svg)](https://coveralls.io/r/sagiegurari/simple-web-notification) [![Known Vulnerabilities](https://snyk.io/test/github/sagiegurari/simple-web-notification/badge.svg)](https://snyk.io/test/github/sagiegurari/simple-web-notification) [![Inline docs](http://inch-ci.org/github/sagiegurari/simple-web-notification.svg?branch=master)](http://inch-ci.org/github/sagiegurari/simple-web-notification) [![License](https://img.shields.io/npm/l/simple-web-notification.svg?style=flat)](https://github.com/sagiegurari/simple-web-notification/blob/master/LICENSE)
+[![NPM Version](http://img.shields.io/npm/v/simple-web-notification.svg?style=flat)](https://www.npmjs.org/package/simple-web-notification) [![CI](https://github.com/sagiegurari/simple-web-notification/workflows/CI/badge.svg?branch=master)](https://github.com/sagiegurari/simple-web-notification/actions) [![Coverage Status](https://coveralls.io/repos/sagiegurari/simple-web-notification/badge.svg)](https://coveralls.io/r/sagiegurari/simple-web-notification) [![Known Vulnerabilities](https://snyk.io/test/github/sagiegurari/simple-web-notification/badge.svg)](https://snyk.io/test/github/sagiegurari/simple-web-notification) [![Inline docs](http://inch-ci.org/github/sagiegurari/simple-web-notification.svg?branch=master)](http://inch-ci.org/github/sagiegurari/simple-web-notification) [![License](https://img.shields.io/npm/l/simple-web-notification.svg?style=flat)](https://github.com/sagiegurari/simple-web-notification/blob/master/LICENSE)
 
 > Web Notifications made easy
 
@@ -142,6 +142,7 @@ See [contributing guide](.github/CONTRIBUTING.md)
 
 | Date        | Version | Description |
 | ----------- | ------- | ----------- |
+| 2020-05-11  | v2.0.0  | Migrate to github actions, upgrade minimal node version and remove bower |
 | 2019-02-08  | v1.0.32 | Maintenance |
 | 2018-06-25  | v1.0.28 | Expose webNotification.requestPermission #5 |
 | 2018-06-14  | v1.0.26 | Better error detection on chrome mobile #4 |
diff --git a/bower.json b/bower.json
diff --git a/docs/CHANGELOG.md b/docs/CHANGELOG.md
@@ -1,5 +1,6 @@
 | Date        | Version | Description |
 | ----------- | ------- | ----------- |
+| 2020-05-11  | v2.0.0  | Migrate to github actions, upgrade minimal node version and remove bower |
 | 2019-02-08  | v1.0.32 | Maintenance |
 | 2018-06-25  | v1.0.28 | Expose webNotification.requestPermission #5 |
 | 2018-06-14  | v1.0.26 | Better error detection on chrome mobile #4 |
diff --git a/docs/example/example.js b/docs/example/example.js
@@ -1,17 +1,17 @@
 /*global console: false */
 
 document.addEventListener('DOMContentLoaded', function onLoad() {
-    var serviceWorkerRegistration;
+    let serviceWorkerRegistration;
 
     if (navigator.serviceWorker) {
         navigator.serviceWorker.register('service-worker.js').then(function (registration) {
             serviceWorkerRegistration = registration;
         });
     }
 
-    var titleElement = document.getElementById('title');
-    var messageElement = document.getElementById('message');
-    var buttonElement = document.querySelector('.btn');
+    const titleElement = document.getElementById('title');
+    const messageElement = document.getElementById('message');
+    const buttonElement = document.querySelector('.btn');
 
     titleElement.value = 'Example Notification';
     messageElement.value = 'This is some notification text.';
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "simple-web-notification",
-  "version": "1.0.32",
+  "version": "2.0.0",
   "description": "Web Notifications made easy.",
   "author": {
     "name": "Sagie Gur-Ari",
diff --git a/project/config/README-template.md b/project/config/README-template.md
@@ -1,6 +1,6 @@
 # {"gitdown": "gitinfo", "name": "name"}
 
-[![NPM Version](http://img.shields.io/npm/v/{"gitdown": "gitinfo", "name": "name"}.svg?style=flat)](https://www.npmjs.org/package/{"gitdown": "gitinfo", "name": "name"}) [![Build Status](https://travis-ci.org/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}.svg)](http://travis-ci.org/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}) [![Coverage Status](https://coveralls.io/repos/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}/badge.svg)](https://coveralls.io/r/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}) [![Known Vulnerabilities](https://snyk.io/test/github/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}/badge.svg)](https://snyk.io/test/github/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}) [![Inline docs](http://inch-ci.org/github/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}.svg?branch=master)](http://inch-ci.org/github/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}) [![License](https://img.shields.io/npm/l/{"gitdown": "gitinfo", "name": "name"}.svg?style=flat)](https://github.com/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}/blob/master/LICENSE)
+[![NPM Version](http://img.shields.io/npm/v/{"gitdown": "gitinfo", "name": "name"}.svg?style=flat)](https://www.npmjs.org/package/{"gitdown": "gitinfo", "name": "name"}) [![CI](https://github.com/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}/workflows/CI/badge.svg?branch=master)](https://github.com/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}/actions) [![Coverage Status](https://coveralls.io/repos/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}/badge.svg)](https://coveralls.io/r/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}) [![Known Vulnerabilities](https://snyk.io/test/github/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}/badge.svg)](https://snyk.io/test/github/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}) [![Inline docs](http://inch-ci.org/github/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}.svg?branch=master)](http://inch-ci.org/github/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}) [![License](https://img.shields.io/npm/l/{"gitdown": "gitinfo", "name": "name"}.svg?style=flat)](https://github.com/{"gitdown": "gitinfo", "name": "username"}/{"gitdown": "gitinfo", "name": "name"}/blob/master/LICENSE)
 
 > Web Notifications made easy
 
diff --git a/project/config/jshintrc-common.json b/project/config/jshintrc-common.json
@@ -2,6 +2,7 @@
   "bitwise": true,
   "curly": true,
   "eqeqeq": true,
+  "esversion": 6,
   "forin": true,
   "freeze": true,
   "futurehostile": true,
diff --git a/project/config/karma.conf.js b/project/config/karma.conf.js
@@ -3,8 +3,8 @@
 module.exports = function (config) {
     'use strict';
 
-    var mainJSFile = require('../../bower.json').main;
-    var commons = require('js-project-commons');
+    const mainJSFile = require('../../package.json').main;
+    const commons = require('js-project-commons');
 
     commons.tools.karma(config, {
         files: [
diff --git a/stylelint.config.js b/stylelint.config.js
@@ -1,5 +1,5 @@
 'use strict';
 
-var commons = require('js-project-commons');
+const commons = require('js-project-commons');
 
 module.exports = commons.lint.stylelint;
diff --git a/test/helpers/notify-mock.js b/test/helpers/notify-mock.js
@@ -1,21 +1,21 @@
 window.Notification = (function Notification() {
     'use strict';
 
-    var noop = function () {
+    const noop = function () {
             return undefined;
     };
 
-    var permissionInfo = {
+    const permissionInfo = {
         value: null
     };
 
-    var oncePermission;
+    let oncePermission;
 
-    var Lib = function (title, options) {
-        var validateNotification = Lib.validateNotification || noop;
+    const Lib = function (title, options) {
+        const validateNotification = Lib.validateNotification || noop;
         validateNotification(title, options);
 
-        var self = this;
+        const self = this;
         self.close = function () {
             if (options.onClick) {
                 self.onclick();
@@ -27,7 +27,7 @@ window.Notification = (function Notification() {
 
     Object.defineProperty(Lib, 'permission', {
         enumerable: true,
-        get: function () {
+        get() {
             return permissionInfo.value;
         }
     });
diff --git a/test/spec/web-notification-spec.js b/test/spec/web-notification-spec.js
diff --git a/web-notification.js b/web-notification.js

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "simple-web-notification",`
`3`		`- "version": "1.0.32",`
	`3`	`+ "version": "2.0.0",`
`4`	`4`	`"description": "Web Notifications made easy.",`
`5`	`5`	`"author": {`
`6`	`6`	`"name": "Sagie Gur-Ari",`