Merge #155

155: Remove guaranteed compatibility with versions to 10> and <15 r=bidoubiwa a=bidoubiwa

fixes #154 

needs `tests` to be removed from mandatory tests in settings @curquiza 

Since the builded package can be used (without guarantee) in other node versions, we should not make it mandatory to use the right node version. It is at the discretion of the user. 

It is how most libraries handle node support. The use of `engines` in packages json limits the possibility to install the package in a specific node environment (see [issue](meilisearch/documentation#963 (comment))).
Whereas typically a package that is bundled and transpiled should be as compatible as possible with all js environments. But, the packages we use in development are not. This is why we add it in the CONTRIBUTING guide


Co-authored-by: Charlotte Vermandel <[email protected]>
Co-authored-by: cvermand <[email protected]>

Author: bors[bot]

.github/workflows/tests.yml

@@ -11,14 +11,22 @@ on:
 
 jobs:
   tests:
-    name: tests
     runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        node: [ '12', '14' ]
+    name: Tests (Node.js ${{ matrix.node }})
     steps:
-    - uses: actions/checkout@v2
-    - name: Install dependencies
-      run: yarn install
-    - name: Run tests
-      run: yarn run test
+      - uses: actions/checkout@v2
+      - name: Setup node
+        uses: actions/setup-node@v1
+        with:
+          node-version: ${{ matrix.node }}
+      - name: Install dependencies
+        run: yarn install
+      - name: Run tests
+        run: yarn run test
 
   linter:
     name: linter-check

CONTRIBUTING.md

@@ -26,6 +26,13 @@ First of all, thank you for contributing to MeiliSearch! The goal of this docume
 
 ## Development Workflow
 
+### Requirements <!-- omit in TOC -->
+
+To run this project, you will need:
+
+- Node.js >= v12 and node < 15
+- Yarn
+
 ### Setup <!-- omit in TOC -->
 
 ```bash

bors.toml

@@ -1,3 +1,7 @@
-status = ['tests', 'linter-check']
+status = [
+    'linter-check',
+    'Tests (Node.js 12)',
+    'Tests (Node.js 14)'
+]
 # 1 hour timeout
 timeout-sec = 3600