Skip to content

Commit

Permalink
Merge pull request #99 from batoulapps/98-modernize-bundling
Browse files Browse the repository at this point in the history
Modernize bundling
  • Loading branch information
sgtsquiggs authored Apr 19, 2022
2 parents 8246e67 + e5c07b4 commit c15ba6e
Show file tree
Hide file tree
Showing 17 changed files with 8,468 additions and 17,097 deletions.
2 changes: 2 additions & 0 deletions .eslintignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
/lib
/Shared
9 changes: 8 additions & 1 deletion .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,12 @@ module.exports = {
sourceType: 'module',
},

extends: ['eslint:recommended', 'plugin:jest/recommended'],
extends: [
'eslint:recommended',
'plugin:jest/recommended',
'plugin:jest/style',
'plugin:prettier/recommended',
],

env: {
node: true,
Expand All @@ -21,4 +26,6 @@ module.exports = {
'prefer-const': 'error',
'object-shorthand': 'error',
},

plugins: ['jest'],
};
15 changes: 6 additions & 9 deletions .github/workflows/node.js.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ jobs:

strategy:
matrix:
node-version: [10.x, 12.x, 14.x, 15.x]
node-version: [12.x, 14.x, 16.x, 17.x]

steps:
- uses: actions/checkout@v2
Expand All @@ -28,9 +28,6 @@ jobs:
- run: npm install -g npm@7 # update to npm 7
- run: npm ci # equivalent of npm ci
- run: npm run build
- run: ./scripts/ci-build-check.sh
env:
CI: true

# testing job
test:
Expand All @@ -39,7 +36,7 @@ jobs:

strategy:
matrix:
node-version: [10.x, 12.x, 14.x, 15.x]
node-version: [12.x, 14.x, 16.x, 17.x]
steps:
- uses: actions/checkout@v2
- name: Testing on node ${{ matrix.node-version }} 🧪
Expand All @@ -48,6 +45,7 @@ jobs:
node-version: ${{ matrix.node-version }}
- run: npm install -g npm@7 # update to npm 7
- run: npm ci
- run: npm run lint
- run: npm test

# coverage uploading job
Expand All @@ -61,9 +59,9 @@ jobs:
- uses: actions/setup-node@v1
with:
# latest nodejs LTS
node-version: 14.x
- run: npm install -g npm@7 # update to npm 7
node-version: 16.x
- run: npm ci
- run: npm run lint
- run: npm test
- name: Upload coverage to Codecov
uses: codecov/codecov-action@v1
Expand All @@ -88,8 +86,7 @@ jobs:
- uses: actions/setup-node@v1
with:
# latest nodejs LTS
node-version: 14.x
- run: npm install -g npm@7 # update to npm 7
node-version: 16.x
- run: npm ci
- name: Preparing a release for npm and github 🚀
# more about this action here: https://github.com/cycjimmy/semantic-release-action
Expand Down
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,4 @@ coverage/
.idea
.vscode/
.env
lib/
16 changes: 11 additions & 5 deletions .npmignore
Original file line number Diff line number Diff line change
@@ -1,8 +1,14 @@
test/
.github/
.husky/
coverage/
scripts/
Shared/
src/
test/
coverage/
*.config.js
.nvmrc
.eslint*
.prettier*
.releaserc.json
example.html
.travis.yml
jest.config.js
babel.config.js
webpack.config.js
3 changes: 1 addition & 2 deletions .prettierignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,2 @@
/Adhan.js
/Adhan.js.map
/lib
/Shared
Loading

0 comments on commit c15ba6e

Please sign in to comment.