Revert "ci: swap deprecated xvfb Action"

.github/workflows/main.yaml

@@ -32,10 +32,14 @@ jobs:
       - name: Test Syntax Highlighting
         run: npm run test:grammar
       - name: Test Unittests
-        run: xvfb-run --auto-servernum npm run test
+        uses: GabrielBB/xvfb-action@v1
+        with:
+          run: npm run test
       # This will not fail the job if tests fail so we have to npm test separately
       - name: Coverage report
-        run: xvfb-run --auto-servernum npm run coverage
+        uses: GabrielBB/xvfb-action@v1
+        with:
+          run: npm run coverage
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v3
         with:

package.json

@@ -251,7 +251,7 @@
           },
           "fortran.linter.initialize": {
             "type": "boolean",
-            "default": false,
+            "default": true,
             "markdownDescription": "Attempt to initialize the linter by mock-compiling all files in the workspace.",
             "order": 5
           },

src/lint/provider.ts

@@ -341,15 +341,12 @@ export class FortranLintingProvider {
       // Discard the previous diagnostic state for this document
       if (this.fortranDiagnostics.has(document.uri)) this.fortranDiagnostics.delete(document.uri);
       this.logger.debug('[lint] No linting diagnostics to show');
-      console.log('[lint] No linting diagnostics to show');
       return;
     }
     let diagnostics: vscode.Diagnostic[] = this.linter.parse(output);
-    console.log('[lint] Parsing output to vscode.Diagnostics', diagnostics);
     this.logger.debug('[lint] Parsing output to vscode.Diagnostics', diagnostics);
     // Remove duplicates from the diagnostics array
     diagnostics = [...new Map(diagnostics.map(v => [JSON.stringify(v), v])).values()];
-    console.log('[lint] Removing duplicates. vscode.Diagnostics are now:', diagnostics);
     this.logger.debug('[lint] Removing duplicates. vscode.Diagnostics are now:', diagnostics);
     this.fortranDiagnostics.set(document.uri, diagnostics);
     return diagnostics;
@@ -380,8 +377,6 @@ export class FortranLintingProvider {
       `[build.single] compiler: "${this.settings.compiler}" located in: "${command}"`
     );
     this.logger.info(`[build.single] Compiler query command line: ${command} ${argList.join(' ')}`);
-    console.log(`[build.single] compiler: "${this.settings.compiler}" located in: "${command}"`);
-    console.log(`[build.single] Compiler query command line: ${command} ${argList.join(' ')}`);
 
     try {
       const fypp = await this.getFyppProcess(document);
@@ -397,7 +392,6 @@ export class FortranLintingProvider {
         );
         const output: string = stdout + stderr;
         this.logger.debug(`[build.single] Compiler output:\n${output}`);
-        console.log(`[build.single] Compiler output:\n${output}`);
         return output;
       } catch (err) {
         this.logger.error(`[build.single] Compiler error:`, err);

tsconfig.json

@@ -8,8 +8,8 @@
     "resolveJsonModule": true,
     "esModuleInterop": true,
     "removeComments": true,
-    "rootDir": "."
+    "rootDir": ".",
   },
   "include": ["src/**/*.ts", "src/**/*.json"],
-  "exclude": ["node_modules", ".vscode-test", "assets/videos"]
+  "exclude": ["node_modules", ".vscode-test", "assets/videos"],
 }