Utilize async/await

Simplify the flow of the code quite a bit by using async/await.

Author: Arcanemagus

lib/index.js

@@ -2,6 +2,12 @@
 
 // eslint-disable-next-line import/extensions, import/no-extraneous-dependencies
 import { CompositeDisposable } from 'atom';
+import fs from 'fs';
+import path from 'path';
+import { findAsync, rangeFromLineNumber } from 'atom-linter';
+import stripJSONComments from 'strip-json-comments';
+
+const readFile = require('tiny-promisify')(fs.readFile);
 
 const grammarScopes = [];
 
@@ -23,55 +29,44 @@ export function deactivate() {
   subscriptions.dispose();
 }
 
-function getConfig(filePath) {
-  const fs = require('fs');
-  const path = require('path');
-  const readFile = require('tiny-promisify')(fs.readFile);
-  const { findAsync } = require('atom-linter');
-
-  return findAsync(path.dirname(filePath), '.htmlhintrc')
-    .then((configPath) => {
-      if (configPath) {
-        return readFile(configPath, 'utf8');
-      }
-      return null;
-    })
-    .then((conf) => {
-      if (conf) {
-        return JSON.parse(require('strip-json-comments')(conf));
-      }
-      return null;
-    });
-}
+const getConfig = async (filePath) => {
+  const configPath = await findAsync(path.dirname(filePath), '.htmlhintrc');
+  let conf = null;
+  if (configPath !== null) {
+    conf = await readFile(configPath, 'utf8');
+  }
+  if (conf) {
+    return JSON.parse(stripJSONComments(conf));
+  }
+  return null;
+};
 
 export function provideLinter() {
   return {
     name: 'htmlhint',
     grammarScopes,
     scope: 'file',
     lintOnFly: true,
-    lint: (editor) => {
+    lint: async (editor) => {
       const { HTMLHint } = require('htmlhint');
 
-      const text = editor.getText();
+      const fileText = editor.getText();
       const filePath = editor.getPath();
 
-      if (!text) {
-        return Promise.resolve([]);
+      if (!fileText) {
+        return [];
       }
 
-      return getConfig(filePath)
-        .then(ruleset => HTMLHint.verify(text, ruleset || undefined))
-        .then((messages) => {
-          const { rangeFromLineNumber } = require('atom-linter');
+      const ruleset = await getConfig(filePath);
+
+      const messages = HTMLHint.verify(fileText, ruleset || undefined);
 
-          return messages.map(message => ({
-            range: rangeFromLineNumber(editor, message.line - 1, message.col - 1),
-            type: message.type,
-            text: message.message,
-            filePath
-          }));
-        });
+      return messages.map(message => ({
+        range: rangeFromLineNumber(editor, message.line - 1, message.col - 1),
+        type: message.type,
+        text: message.message,
+        filePath
+      }));
     }
   };
 }