refactor: sort imports

Author: gnikit

.eslintrc.js

@@ -11,5 +11,14 @@ module.exports = {
     '@typescript-eslint/no-explicit-any': 0,
     '@typescript-eslint/explicit-module-boundary-types': 0,
     '@typescript-eslint/no-non-null-assertion': 0,
+
+    'import/order': [
+      'error',
+      {
+        'alphabetize': { order: 'asc' },
+        'newlines-between': 'always',
+        'groups': ['builtin', 'external', 'internal', 'parent', 'sibling', 'index'],
+      },
+    ],
   },
 };

src/extension.ts

@@ -1,19 +1,21 @@
 'use strict';
 
+import * as fs from 'fs';
+import * as path from 'path';
+
 import * as vscode from 'vscode';
+
 import * as pkg from '../package.json';
-import * as path from 'path';
-import * as fs from 'fs';
 
+import { WhatsNew } from './commands/commands';
 import { FortranCompletionProvider } from './fallback-features/completion-provider';
 import { FortranDocumentSymbolProvider } from './fallback-features/document-symbol-provider';
-import { FortranFormattingProvider } from './format/formatting-provider';
-import { FortlsClient } from './lsp/client';
 import { FortranHoverProvider } from './fallback-features/hover-provider';
-import { FortranLintingProvider } from './lint/linter-provider';
-import { EXTENSION_ID, FortranDocumentSelector } from './util/tools';
+import { FortranFormattingProvider } from './format/provider';
+import { FortranLintingProvider } from './lint/provider';
+import { FortlsClient } from './lsp/client';
 import { getConfigLogLevel, Logger } from './services/logging';
-import { WhatsNew } from './commands/commands';
+import { EXTENSION_ID, FortranDocumentSelector } from './util/tools';
 
 // Make it global to catch errors when activation fails
 const logger = new Logger(

src/fallback-features/completion-provider.ts

@@ -1,10 +1,11 @@
-import { CancellationToken, TextDocument, Position, Hover } from 'vscode';
-import * as fs from 'fs';
 import * as vscode from 'vscode';
-import { isPositionInString, FORTRAN_KEYWORDS } from '../util/helper';
-import { getDeclaredFunctions } from './functions';
-import { EXTENSION_ID } from '../util/tools';
+import { Position, TextDocument } from 'vscode';
+
 import { Logger } from '../services/logging';
+import { FORTRAN_KEYWORDS, isPositionInString } from '../util/helper';
+import { EXTENSION_ID } from '../util/tools';
+
+import { getDeclaredFunctions } from './functions';
 import intrinsics from './intrinsics.json';
 
 class CaseCoverter {

src/fallback-features/document-symbol-provider.ts

@@ -1,6 +1,6 @@
 import { CancellationToken, TextDocument, TextLine, SymbolInformation } from 'vscode';
-
 import * as vscode from 'vscode';
+
 import {
   parseFunction as getDeclaredFunction,
   parseSubroutine as getDeclaredSubroutine,

src/fallback-features/hover-provider.ts

@@ -1,5 +1,7 @@
 import { CancellationToken, TextDocument, Position, Hover } from 'vscode';
+
 import { Logger } from '../services/logging';
+
 import intrinsics from './intrinsics.json';
 
 export class FortranHoverProvider {

src/fallback-features/variables.ts

@@ -1,5 +1,6 @@
 /* c8 ignore start */
 import * as vscode from 'vscode';
+
 import { Variable } from './functions';
 
 const varibleDecRegEx =

src/format/formatting-provider.ts renamed to src/format/provider.ts

@@ -1,8 +1,9 @@
 'use strict';
 
 import * as path from 'path';
-import which from 'which';
+
 import * as vscode from 'vscode';
+import which from 'which';
 
 import { Logger } from '../services/logging';
 import {

src/lint/linter-provider.ts renamed to src/lint/provider.ts

@@ -1,16 +1,26 @@
 'use strict';
 
+import * as cp from 'child_process';
 import * as fs from 'fs';
 import * as path from 'path';
-import * as cp from 'child_process';
-import which from 'which';
+
+import { glob } from 'glob';
 import * as semver from 'semver';
 import * as vscode from 'vscode';
-import { glob } from 'glob';
+import which from 'which';
 
 import * as pkg from '../../package.json';
+import {
+  BuildDebug,
+  BuildRun,
+  InitLint,
+  CleanLintFiles,
+  RescanLint,
+  CleanLintDiagnostics,
+} from '../commands/commands';
 import { Logger } from '../services/logging';
-import { GNULinter, GNUModernLinter, IntelLinter, LFortranLinter, NAGLinter } from './linters';
+import { GlobPaths } from '../util/glob-paths';
+import { arraysEqual } from '../util/helper';
 import {
   EXTENSION_ID,
   resolveVariables,
@@ -20,16 +30,8 @@ import {
   isFortran,
   shellTask,
 } from '../util/tools';
-import { arraysEqual } from '../util/helper';
-import { GlobPaths } from '../util/glob-paths';
-import {
-  BuildDebug,
-  BuildRun,
-  InitLint,
-  CleanLintFiles,
-  RescanLint,
-  CleanLintDiagnostics,
-} from '../commands/commands';
+
+import { GNULinter, GNUModernLinter, IntelLinter, LFortranLinter, NAGLinter } from './linters';
 
 const GNU = new GNULinter();
 const GNU_NEW = new GNUModernLinter();

src/lsp/client.ts

@@ -1,11 +1,15 @@
 'use strict';
 
+import { spawnSync } from 'child_process';
 import * as os from 'os';
 import * as path from 'path';
+
 import * as vscode from 'vscode';
-import { spawnSync } from 'child_process';
 import { commands, window, workspace, TextDocument } from 'vscode';
 import { LanguageClient, LanguageClientOptions, ServerOptions } from 'vscode-languageclient/node';
+
+import { RestartLS } from '../commands/commands';
+import { Logger } from '../services/logging';
 import {
   EXTENSION_ID,
   FortranDocumentSelector,
@@ -15,8 +19,6 @@ import {
   pipInstall,
   resolveVariables,
 } from '../util/tools';
-import { Logger } from '../services/logging';
-import { RestartLS } from '../commands/commands';
 
 // The clients are non member variables of the class because they need to be
 // shared for command registration. The command operates on the client and not

src/services/logging.ts

@@ -1,4 +1,5 @@
 import { OutputChannel, window, workspace, WorkspaceConfiguration } from 'vscode';
+
 import { EXTENSION_ID } from '../util/tools';
 
 export enum LogLevel {