Merge pull request #12 from fewieden/develop

v2

.codeclimate.yml

@@ -5,10 +5,10 @@ engines:
     enabled: true
     config:
       languages:
-      - javascript
+        - javascript
   eslint:
     enabled: true
-    channel: "eslint-3"
+    channel: "eslint-6"
     checks:
       import/no-unresolved:
         enabled: false
@@ -18,9 +18,9 @@ engines:
       enabled: true
 ratings:
   paths:
-  - "**.js"
-  - "**.css"
-  - "**.md"
-exclude_paths: [
-  "node_modules/**/*"
-]
+    - "**.js"
+    - "**.css"
+    - "**.md"
+exclude_paths:
+  - "node_modules/**/*"
+  - "docs/**/*"

.eslintignore

@@ -0,0 +1 @@
+docs/*

.eslintrc

@@ -1,18 +1,34 @@
 {
-    "extends": "airbnb-base",
-    "rules": {
-        "comma-dangle": 0,
-        "indent": [2, 4],
-        "max-len": [2, 120, { "ignoreStrings": true }],
-        "radix": [2, "as-needed"],
-        "no-console": 0
+    "extends": ["esnext", "esnext/style-guide", "node", "node/style-guide"],
+    "parserOptions": {
+        "ecmaVersion": 2017,
+        "sourceType": "module"
     },
     "settings": {
         "import/core-modules": [ "node_helper" ]
     },
     "env": {
         "browser": true,
         "node": true,
-        "es6": true
+        "es6": true,
+        "jest": true
+    },
+    "globals": {
+        "Module": true,
+        "Log": true
+    },
+    "rules": {
+        "import/no-commonjs": 0,
+        "import/no-nodejs-modules": 0,
+        "semi": 0,
+        "comma-dangle": 0,
+        "indent": ["error", 4],
+        "template-curly-spacing": 0,
+        "no-console": 0,
+        "curly": ["error", "all"],
+        "array-bracket-spacing": 0,
+        "space-before-function-paren": 0,
+        "object-curly-spacing": ["error", "never"],
+        "object-property-newline": ["error", { "allowAllPropertiesOnSameLine": true }]
     }
 }

.github/CONTRIBUTING.md

@@ -0,0 +1,8 @@
+# Contribution Guidelines
+
+Thanks for contributing to this module!
+
+Please create pull requests to the branch `develop`.
+
+To hold one code style and standard there are several linters and tools in this project set. Make sure you fulfill the requirements.
+Also, there will be automatically analysis performed once you created the pull request.

.github/FUNDING.yml

@@ -0,0 +1 @@
+custom: ['https://paypal.me/fewieden']

.github/ISSUE_TEMPLATE.md

@@ -0,0 +1,9 @@
+Platform (Hardware/OS):
+
+Node version:
+
+MagicMirror version:
+
+Module version:
+
+Description of the issue:

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,5 @@
+Please create pull requests to the branch `develop`.
+
+* Does the pull request solve an issue (add a reference)?
+* What are the features of this pr?
+* Add screenshots for visual changes.

.github/workflows/build.yml

@@ -0,0 +1,24 @@
+# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
+
+name: build
+
+on:
+  push:
+    branches: [ master, develop ]
+  pull_request:
+    branches: [ master, develop ]
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        node-version: [ 14.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/changelog.yml

@@ -0,0 +1,15 @@
+name: changelog
+
+on:
+  pull_request:
+    types: [opened, synchronize, reopened, ready_for_review, labeled, unlabeled]
+
+jobs:
+  check:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: dangoslen/changelog-enforcer@v1.6.1
+        with:
+          changeLogPath: CHANGELOG.md
+          skipLabels: Skip Changelog

.gitignore

@@ -0,0 +1,2 @@
+/node_modules/**/*
+docs/

.mdlrc

@@ -1,2 +1,2 @@
 all
-rules "~MD013", "~MD026", "~MD033"
+rules "~MD013", "~MD024", "~MD026", "~MD033"

CHANGELOG.md

@@ -1,5 +1,36 @@
 # MMM-NHL Changelog
 
+## [2.0.0]
+
+### Added
+
+* Nunjuck templates
+* French translations thanks to [matlem037](https://github.com/matlem037)
+* Dependency `node-fetch`
+* Config option `daysInPast`
+* Config option `daysAhead`
+* Config option `liveReloadInterval` thanks to [dannoh](https://github.com/dannoh).
+* Config option `showNames` thanks to [dannoh](https://github.com/dannoh).
+* Config option `showLogos` thanks to [dannoh](https://github.com/dannoh).
+* Support for game status `postponed` thanks to [dannoh](https://github.com/dannoh).
+* Github actions (linting and changelog enforcer)
+* JSDoc documentation
+
+### Changed
+
+* Switched API for data feed.
+* Display logos from remote.
+* Retrieve team list from API.
+* ESLint recommended instead of airbnb ruleset.
+
+### Removed
+
+* Config option `format`, instead rendering information based on locale.
+* Travis integration
+* Dependency `moment-timezone`
+* Dependency `request`
+* Local team logos
+
 ## [1.0.1]
 
 ### Added

MMM-NHL.css

@@ -5,6 +5,11 @@
 }
 
 .MMM-NHL .icon {
+    width: 37px;
+    height: 25px;
+}
+
+.MMM-NHL .no-color {
     -webkit-filter: grayscale(100%);
     filter: grayscale(100%);
 }