Merge pull request #40 from openlayers/no-elseif-return

No else-if return

Author: tschaub

.github/dependabot.yml

@@ -0,0 +1,12 @@
+version: 2
+updates:
+- package-ecosystem: npm
+  directory: "/"
+  schedule:
+    interval: weekly
+  open-pull-requests-limit: 10
+  versioning-strategy: increase-if-necessary
+- package-ecosystem: github-actions
+  directory: "/"
+  schedule:
+    interval: weekly

.github/workflows/test.yml

@@ -10,31 +10,22 @@ on:
 
 jobs:
   test:
-    name: ${{ matrix.platform }}
-    runs-on: ${{ matrix.platform }}
-
-    strategy:
-      fail-fast: false
-      matrix:
-        platform:
-          - ubuntu-latest
-          - macos-latest
-          - windows-latest
-        node:
-          - 16
+    name: Test
+    runs-on: ubuntu-latest
 
     steps:
-      - name: Set git to use LF
-        run: |
-          git config --global core.autocrlf false
-          git config --global core.eol lf
+      - name: Clone Repository
+        uses: actions/checkout@v3
 
-      - uses: actions/checkout@v2
-      - name: Use Node.js ${{ matrix.node }}
-        uses: actions/setup-node@v2
+      - name: Setup Node.js
+        uses: actions/setup-node@v3
         with:
-          node-version: ${{ matrix.node }}
-      - run: npm ci
-      - run: npm test
+          node-version: '16'
+
+      - name: Install Dependencies
+        run: npm ci
+
+      - name: Run Tests
+        run: npm test
         env:
           CI: true

index.js

@@ -60,7 +60,7 @@ module.exports = {
     'no-dupe-class-members': 'error',
     'no-dupe-keys': 'error',
     'no-duplicate-case': 'error',
-    'no-else-return': 'error',
+    'no-else-return': ['error', {allowElseIf: false}],
     'no-empty': 'error',
     'no-empty-character-class': 'error',
     'no-eq-null': 'error',