Skip to content

Commit abd4703

Browse files
committed
Migrate to github actions, upgrade min es version and remove bower
1 parent 110f8f2 commit abd4703

16 files changed

+56
-71
lines changed

.eslintrc.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
22

3-
var commons = require('js-project-commons');
3+
const commons = require('js-project-commons');
44

55
module.exports = commons.lint.eslint.web;

.github/CONTRIBUTING.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ Fixed a bug or just want to provided additional functionality?<br>
1212
Simply fork this repository, implement your changes and create a pull request.<br>
1313
Few guidelines regarding pull requests:
1414

15-
* This repository is integrated with travis.ci for continuous integration.<br>
15+
* This repository is integrated with github actions for continuous integration.<br>
1616

1717
Your pull request build must pass (the build will run automatically).<br>
1818
You can run the following command locally to ensure the build will pass:

.github/workflows/ci.yml

+26
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
name: CI
2+
on: [push, pull_request]
3+
env:
4+
CLICOLOR_FORCE: 1
5+
jobs:
6+
ci:
7+
name: CI
8+
runs-on: ubuntu-latest
9+
strategy:
10+
fail-fast: false
11+
steps:
12+
- name: Checkout
13+
uses: actions/checkout@v2
14+
- name: Install
15+
uses: ianwalter/[email protected]
16+
with:
17+
args: npm install
18+
- name: Run CI
19+
uses: ianwalter/[email protected]
20+
with:
21+
args: npm test
22+
- name: Coveralls
23+
uses: coverallsapp/github-action@master
24+
with:
25+
github-token: ${{ secrets.GITHUB_TOKEN }}
26+
path-to-lcov: './target/coverage/report/lcov.info'

.travis.yml

-10
This file was deleted.

Gruntfile.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
/*eslint-disable no-underscore-dangle*/
66

77
module.exports = function (grunt) {
8-
var commons = require('js-project-commons');
8+
const commons = require('js-project-commons');
99

1010
commons.grunt.config.initConfig(grunt, {
1111
buildConfig: {

README.md

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# angular-web-notification
22

3-
[![NPM Version](http://img.shields.io/npm/v/angular-web-notification.svg?style=flat)](https://www.npmjs.org/package/angular-web-notification) [![Build Status](https://travis-ci.org/sagiegurari/angular-web-notification.svg)](http://travis-ci.org/sagiegurari/angular-web-notification) [![Coverage Status](https://coveralls.io/repos/sagiegurari/angular-web-notification/badge.svg)](https://coveralls.io/r/sagiegurari/angular-web-notification) [![Known Vulnerabilities](https://snyk.io/test/github/sagiegurari/angular-web-notification/badge.svg)](https://snyk.io/test/github/sagiegurari/angular-web-notification) [![Inline docs](http://inch-ci.org/github/sagiegurari/angular-web-notification.svg?branch=master)](http://inch-ci.org/github/sagiegurari/angular-web-notification) [![License](https://img.shields.io/npm/l/angular-web-notification.svg?style=flat)](https://github.com/sagiegurari/angular-web-notification/blob/master/LICENSE)
3+
[![NPM Version](http://img.shields.io/npm/v/angular-web-notification.svg?style=flat)](https://www.npmjs.org/package/angular-web-notification) [![CI](https://github.com/sagiegurari/angular-web-notification/workflows/CI/badge.svg?branch=master)](https://github.com/sagiegurari/angular-web-notification/actions) [![Coverage Status](https://coveralls.io/repos/sagiegurari/angular-web-notification/badge.svg)](https://coveralls.io/r/sagiegurari/angular-web-notification) [![Known Vulnerabilities](https://snyk.io/test/github/sagiegurari/angular-web-notification/badge.svg)](https://snyk.io/test/github/sagiegurari/angular-web-notification) [![Inline docs](http://inch-ci.org/github/sagiegurari/angular-web-notification.svg?branch=master)](http://inch-ci.org/github/sagiegurari/angular-web-notification) [![License](https://img.shields.io/npm/l/angular-web-notification.svg?style=flat)](https://github.com/sagiegurari/angular-web-notification/blob/master/LICENSE)
44

55
> Web Notifications AngularJS Service
66
@@ -153,6 +153,7 @@ See [contributing guide](.github/CONTRIBUTING.md)
153153

154154
| Date | Version | Description |
155155
| ----------- | ------- | ----------- |
156+
| 2020-05-11 | v2.0.0 | Migrate to github actions, upgrade minimal node version and remove bower |
156157
| 2019-02-08 | v1.2.31 | Maintenance |
157158
| 2017-08-25 | v1.2.24 | Document support of service worker web notifications |
158159
| 2017-01-22 | v1.2.0 | Split the internal web notification API into a new project: simple-web-notification |

angular-web-notification.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
(function initWebNotification(webNotificationAPI) {
1414
'use strict';
1515

16-
var webNotification = window.angular.module('angular-web-notification', []);
16+
const webNotification = window.angular.module('angular-web-notification', []);
1717

1818
/**
1919
* @ngdoc service
@@ -97,7 +97,7 @@
9797
* });
9898
* ```
9999
*/
100-
var showNotification = webNotificationAPI.showNotification;
100+
const showNotification = webNotificationAPI.showNotification;
101101

102102
if (showNotification) {
103103
return webNotificationAPI;

bower.json

-37
This file was deleted.

docs/CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
| Date | Version | Description |
22
| ----------- | ------- | ----------- |
3+
| 2020-05-11 | v2.0.0 | Migrate to github actions, upgrade minimal node version and remove bower |
34
| 2019-02-08 | v1.2.31 | Maintenance |
45
| 2017-08-25 | v1.2.24 | Document support of service worker web notifications |
56
| 2017-01-22 | v1.2.0 | Split the internal web notification API into a new project: simple-web-notification |

package.json

+4-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "angular-web-notification",
3-
"version": "1.2.31",
3+
"version": "2.0.0",
44
"description": "AngularJS service for displaying web notifications.",
55
"author": {
66
"name": "Sagie Gur-Ari",
@@ -42,14 +42,17 @@
4242
}
4343
},
4444
"dependencies": {
45+
"angular": "^1",
4546
"simple-web-notification": "latest"
4647
},
4748
"devDependencies": {
49+
"angular-mocks": "^1",
4850
"chai": "latest",
4951
"grunt": "latest",
5052
"grunt-cli": "latest",
5153
"grunt-karma": "latest",
5254
"husky": "latest",
55+
"jquery": "latest",
5356
"js-project-commons": "latest",
5457
"karma": "latest",
5558
"karma-chrome-launcher": "latest",

project/config/README-template.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# {"gitdown": "gitinfo", "name": "name"}
22

3-
[![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)
3+
[![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)
44

55
> Web Notifications AngularJS Service
66

project/config/jshintrc-common.json

+1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
"bitwise": true,
33
"curly": true,
44
"eqeqeq": true,
5+
"esversion": 6,
56
"forin": true,
67
"freeze": true,
78
"futurehostile": true,

project/config/karma.conf.js

+6-6
Original file line numberDiff line numberDiff line change
@@ -3,16 +3,16 @@
33
module.exports = function (config) {
44
'use strict';
55

6-
var mainJSFile = require('../../bower.json').main;
7-
var commons = require('js-project-commons');
6+
const mainJSFile = require('../../package.json').main;
7+
const commons = require('js-project-commons');
88

99
commons.tools.karma(config, {
1010
files: [
11-
'bower_components/jquery/dist/jquery.js',
12-
'bower_components/angular/angular.js',
13-
'bower_components/angular-mocks/angular-mocks.js',
11+
'**/jquery/dist/jquery.js',
12+
'**/angular/angular.js',
13+
'**/angular-mocks/angular-mocks.js',
1414
'test/helpers/**/*.js',
15-
'bower_components/simple-web-notification/web-notification.js',
15+
'**/simple-web-notification/web-notification.js',
1616
mainJSFile,
1717
'test/spec/**/*.js'
1818
]

stylelint.config.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
22

3-
var commons = require('js-project-commons');
3+
const commons = require('js-project-commons');
44

55
module.exports = commons.lint.stylelint;

test/helpers/notify-mock.js

+5-5
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,16 @@
11
window.Notification = (function Notification() {
22
'use strict';
33

4-
var permissionInfo = {
4+
const permissionInfo = {
55
value: null
66
};
77

8-
var oncePermission;
8+
let oncePermission;
99

10-
var Lib = function (title, options) {
10+
const Lib = function (title, options) {
1111
Lib.validateNotification(title, options);
1212

13-
var self = this;
13+
const self = this;
1414
self.close = function () {
1515
if (options.onClick) {
1616
self.onclick();
@@ -22,7 +22,7 @@ window.Notification = (function Notification() {
2222

2323
Object.defineProperty(Lib, 'permission', {
2424
enumerable: true,
25-
get: function () {
25+
get() {
2626
return permissionInfo.value;
2727
}
2828
});

test/spec/angular-web-notification-spec.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
describe('angular-web-notification', function () {
44
'use strict';
55

6-
var validShowValidation = function (error, hide, done) {
6+
const validShowValidation = function (error, hide, done) {
77
assert.isNull(error);
88
assert.isFunction(hide);
99

@@ -14,7 +14,7 @@ describe('angular-web-notification', function () {
1414
}
1515
};
1616

17-
var errorValidation = function (error, hide, done) {
17+
const errorValidation = function (error, hide, done) {
1818
assert.isDefined(error);
1919
assert.isNull(hide);
2020
done();
@@ -23,10 +23,10 @@ describe('angular-web-notification', function () {
2323
beforeEach(window.angular.mock.module('angular-web-notification'));
2424

2525
it('library not defined test', inject(function ($injector) {
26-
var showNotification = window.webNotification.showNotification;
26+
const showNotification = window.webNotification.showNotification;
2727
delete window.webNotification.showNotification;
2828

29-
var errorDetected = false;
29+
let errorDetected = false;
3030
try {
3131
$injector.get('webNotification').$get();
3232
} catch (error) {

0 commit comments

Comments
 (0)