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',