Merge pull request #1069 from fortran-lang/ci/xvfb

ci: swap deprecated xvfb Action

Author: gnikit

.github/workflows/main.yaml

@@ -32,14 +32,10 @@ jobs:
       - name: Test Syntax Highlighting
         run: npm run test:grammar
       - name: Test Unittests
-        uses: GabrielBB/xvfb-action@v1
-        with:
-          run: npm run test
+        run: xvfb-run --auto-servernum npm run test
       # This will not fail the job if tests fail so we have to npm test separately
       - name: Coverage report
-        uses: GabrielBB/xvfb-action@v1
-        with:
-          run: npm run coverage
+        run: xvfb-run --auto-servernum 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": true,
+            "default": false,
             "markdownDescription": "Attempt to initialize the linter by mock-compiling all files in the workspace.",
             "order": 5
           },

src/lint/provider.ts

@@ -341,12 +341,15 @@ 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;
@@ -377,6 +380,8 @@ 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);
@@ -392,6 +397,7 @@ 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"]
 }