Update prettie+eslint to current recommendation (#192)

* Added actual linting check with eslint

* Updated to Node 20 (current LTS)

.github/workflows/pr-review.yml

@@ -20,6 +20,9 @@ jobs:
         run: npm ci
 
       - name: Ensure files follow code style
+        run: npm run prettier:check
+
+      - name: Ensure files pass eslint checks
         run: npm run lint
 
   test:

cypress.config.ts

@@ -1,6 +1,6 @@
 import { defineConfig } from "cypress";
 
-let config = {
+const config = {
   e2e: {
     supportFile: false,
     screenshotsFolder: "cypress/screenshots",

cypress/e2e/send-mail-with-php.msmtp.cy.ts

@@ -10,7 +10,7 @@ describe("Send mail with PHP", () => {
         },
       }).should((response) => {
         expect(response.body.totalRecords).to.eql(1);
-        let mail = response.body.mailItems[0];
+        const mail = response.body.mailItems[0];
         expect(mail.fromAddress).to.eql("[email protected]");
         expect(mail.toAddresses).to.eql(["[email protected]"]);
         expect(mail.body).to.eql("Email sent with PHP mail()\n");

eslint.config.mjs

@@ -0,0 +1,4 @@
+import tseslint from "typescript-eslint";
+import eslintConfigPrettier from "eslint-config-prettier";
+
+export default [...tseslint.configs.recommended, eslintConfigPrettier];