Merge pull request #228 from miqh/fix/197

Resolve violations of eslint-comments/no-use lint

.eslintrc.json

@@ -44,7 +44,6 @@
             "@typescript-eslint/prefer-regexp-exec": "off",
             "@typescript-eslint/require-await": "off",
             "@typescript-eslint/restrict-template-expressions": "off",
-            "eslint-comments/no-use": "off",
             "func-style": "off",
             "github/no-then": "off",
             "import/no-extraneous-dependencies": "off",

lib/codeql.js

@@ -407,6 +407,8 @@ function getExtraOptionsFromEnv(path) {
  *
  * - the special terminal step name '*' in `options` matches all path steps
  * - throws an exception if this conversion is impossible.
+ *
+ * Exported for testing.
  */
 function getExtraOptions(options, path, pathInfo) {
     var _a, _b, _c;

src/codeql.ts

@@ -620,8 +620,10 @@ function getExtraOptionsFromEnv(path: string[]) {
  *
  * - the special terminal step name '*' in `options` matches all path steps
  * - throws an exception if this conversion is impossible.
+ *
+ * Exported for testing.
  */
-export /* exported for testing */ function getExtraOptions(
+export function getExtraOptions(
   options: any,
   path: string[],
   pathInfo: string[]