Ts part 5 (#567)

* ts part 3

* part 5

Author: tclindner

src/rules/description-format.js renamed to src/rules/description-format.ts

@@ -1,12 +1,15 @@
-const LintIssue = require('../LintIssue');
 const {isString} = require('../validators/type');
+import {LintIssue} from '../lint-issue';
+import {RuleType} from '../types/rule-type';
+import {Severity} from '../types/severity';
+import {PackageJson} from 'type-fest';
 
 const lintId = 'description-format';
 const nodeName = 'description';
-const ruleType = 'object';
+export const ruleType = RuleType.Object;
 
 // eslint-disable-next-line complexity
-const lint = (packageJsonData, severity, config) => {
+export const lint = (packageJsonData: PackageJson, severity: Severity, config: any): LintIssue | boolean => {
   if (!packageJsonData.hasOwnProperty(nodeName)) {
     return true;
   }
@@ -45,8 +48,3 @@ const lint = (packageJsonData, severity, config) => {
 
   return true;
 };
-
-module.exports = {
-  lint,
-  ruleType,
-};

src/rules/no-absolute-version-dependencies.js renamed to src/rules/no-absolute-version-dependencies.ts

@@ -1,20 +1,18 @@
 const {areVersionsAbsolute} = require('../validators/dependency-audit');
-const LintIssue = require('../LintIssue');
+import {LintIssue} from '../lint-issue';
+import {RuleType} from '../types/rule-type';
+import {Severity} from '../types/severity';
+import {PackageJson} from 'type-fest';
 
 const lintId = 'no-absolute-version-dependencies';
 const nodeName = 'dependencies';
 const message = 'You are using an invalid version range. Please do not use absolute versions.';
-const ruleType = 'optionalObject';
+export const ruleType = RuleType.OptionalObject;
 
-const lint = (packageJsonData, severity, config) => {
+export const lint = (packageJsonData: PackageJson, severity: Severity, config: any): LintIssue | boolean => {
   if (packageJsonData.hasOwnProperty(nodeName) && areVersionsAbsolute(packageJsonData, nodeName, config)) {
     return new LintIssue(lintId, severity, nodeName, message);
   }
 
   return true;
 };
-
-module.exports = {
-  lint,
-  ruleType,
-};

src/rules/no-absolute-version-devDependencies.js renamed to src/rules/no-absolute-version-devDependencies.ts

@@ -1,20 +1,18 @@
 const {areVersionsAbsolute} = require('../validators/dependency-audit');
-const LintIssue = require('../LintIssue');
+import {LintIssue} from '../lint-issue';
+import {RuleType} from '../types/rule-type';
+import {Severity} from '../types/severity';
+import {PackageJson} from 'type-fest';
 
 const lintId = 'no-absolute-version-devDependencies';
 const nodeName = 'devDependencies';
 const message = 'You are using an invalid version range. Please do not use absolute versions.';
-const ruleType = 'optionalObject';
+export const ruleType = RuleType.OptionalObject;
 
-const lint = (packageJsonData, severity, config) => {
+export const lint = (packageJsonData: PackageJson, severity: Severity, config: any): LintIssue | boolean => {
   if (packageJsonData.hasOwnProperty(nodeName) && areVersionsAbsolute(packageJsonData, nodeName, config)) {
     return new LintIssue(lintId, severity, nodeName, message);
   }
 
   return true;
 };
-
-module.exports = {
-  lint,
-  ruleType,
-};

src/rules/no-archive-dependencies.js renamed to src/rules/no-archive-dependencies.ts

@@ -1,20 +1,18 @@
 const {doVersContainArchiveUrl} = require('../validators/dependency-audit');
-const LintIssue = require('../LintIssue');
+import {LintIssue} from '../lint-issue';
+import {RuleType} from '../types/rule-type';
+import {Severity} from '../types/severity';
+import {PackageJson} from 'type-fest';
 
 const lintId = 'no-archive-dependencies';
 const nodeName = 'dependencies';
 const message = 'You are using dependencies via url to archive file. Please use dependencies from npm.';
-const ruleType = 'optionalObject';
+export const ruleType = RuleType.OptionalObject;
 
-const lint = (packageJsonData, severity, config) => {
+export const lint = (packageJsonData: PackageJson, severity: Severity, config: any): LintIssue | boolean => {
   if (packageJsonData.hasOwnProperty(nodeName) && doVersContainArchiveUrl(packageJsonData, nodeName, config)) {
     return new LintIssue(lintId, severity, nodeName, message);
   }
 
   return true;
 };
-
-module.exports = {
-  lint,
-  ruleType,
-};

src/rules/no-archive-devDependencies.js renamed to src/rules/no-archive-devDependencies.ts

@@ -1,20 +1,18 @@
 const {doVersContainArchiveUrl} = require('../validators/dependency-audit');
-const LintIssue = require('../LintIssue');
+import {LintIssue} from '../lint-issue';
+import {RuleType} from '../types/rule-type';
+import {Severity} from '../types/severity';
+import {PackageJson} from 'type-fest';
 
 const lintId = 'no-archive-devDependencies';
 const nodeName = 'devDependencies';
 const message = 'You are using devDependencies via url to archive file. Please use devDependencies from npm.';
-const ruleType = 'optionalObject';
+export const ruleType = RuleType.OptionalObject;
 
-const lint = (packageJsonData, severity, config) => {
+export const lint = (packageJsonData: PackageJson, severity: Severity, config: any): LintIssue | boolean => {
   if (packageJsonData.hasOwnProperty(nodeName) && doVersContainArchiveUrl(packageJsonData, nodeName, config)) {
     return new LintIssue(lintId, severity, nodeName, message);
   }
 
   return true;
 };
-
-module.exports = {
-  lint,
-  ruleType,
-};

src/rules/no-caret-version-dependencies.js renamed to src/rules/no-caret-version-dependencies.ts

@@ -1,12 +1,15 @@
 const {doVersContainInvalidRange} = require('../validators/dependency-audit');
-const LintIssue = require('../LintIssue');
+import {LintIssue} from '../lint-issue';
+import {RuleType} from '../types/rule-type';
+import {Severity} from '../types/severity';
+import {PackageJson} from 'type-fest';
 
 const lintId = 'no-caret-version-dependencies';
 const nodeName = 'dependencies';
 const message = 'You are using an invalid version range. Please do not use ^.';
-const ruleType = 'optionalObject';
+export const ruleType = RuleType.OptionalObject;
 
-const lint = (packageJsonData, severity, config) => {
+export const lint = (packageJsonData: PackageJson, severity: Severity, config: any): LintIssue | boolean => {
   const rangeSpecifier = '^';
 
   if (
@@ -18,8 +21,3 @@ const lint = (packageJsonData, severity, config) => {
 
   return true;
 };
-
-module.exports = {
-  lint,
-  ruleType,
-};

src/rules/no-caret-version-devDependencies.js renamed to src/rules/no-caret-version-devDependencies.ts

@@ -1,12 +1,15 @@
 const {doVersContainInvalidRange} = require('../validators/dependency-audit');
-const LintIssue = require('../LintIssue');
+import {LintIssue} from '../lint-issue';
+import {RuleType} from '../types/rule-type';
+import {Severity} from '../types/severity';
+import {PackageJson} from 'type-fest';
 
 const lintId = 'no-caret-version-devDependencies';
 const nodeName = 'devDependencies';
 const message = 'You are using an invalid version range. Please do not use ^.';
-const ruleType = 'optionalObject';
+export const ruleType = RuleType.OptionalObject;
 
-const lint = (packageJsonData, severity, config) => {
+export const lint = (packageJsonData: PackageJson, severity: Severity, config: any): LintIssue | boolean => {
   const rangeSpecifier = '^';
 
   if (
@@ -18,8 +21,3 @@ const lint = (packageJsonData, severity, config) => {
 
   return true;
 };
-
-module.exports = {
-  lint,
-  ruleType,
-};

src/rules/no-duplicate-properties.js renamed to src/rules/no-duplicate-properties.ts

@@ -1,12 +1,15 @@
 const Parser = require('../Parser');
 const {findDuplicatePropNames} = require('../validators/property');
-const LintIssue = require('../LintIssue');
+import {LintIssue} from '../lint-issue';
+import {RuleType} from '../types/rule-type';
+import {Severity} from '../types/severity';
+import {PackageJson} from 'type-fest';
 
 const lintId = 'no-duplicate-properties';
 const nodeName = '';
-const ruleType = 'standard';
+export const ruleType = RuleType.Standard;
 
-const lint = (packageJsonData, severity) => {
+export const lint = (packageJsonData: PackageJson, severity: Severity): LintIssue | boolean => {
   /**
    * If we send package json straight to npm-package-json-lint, fallback to empty string.
    * Because we already lose information about duplicate properties.
@@ -22,8 +25,3 @@ const lint = (packageJsonData, severity) => {
 
   return true;
 };
-
-module.exports = {
-  lint,
-  ruleType,
-};

src/rules/no-file-dependencies.js renamed to src/rules/no-file-dependencies.ts

@@ -1,20 +1,18 @@
 const {doVersContainFileUrl} = require('../validators/dependency-audit');
-const LintIssue = require('../LintIssue');
+import {LintIssue} from '../lint-issue';
+import {RuleType} from '../types/rule-type';
+import {Severity} from '../types/severity';
+import {PackageJson} from 'type-fest';
 
 const lintId = 'no-file-dependencies';
 const nodeName = 'dependencies';
 const message = 'You are using dependencies via url to local file. Please use dependencies from npm.';
-const ruleType = 'optionalObject';
+export const ruleType = RuleType.OptionalObject;
 
-const lint = (packageJsonData, severity, config) => {
+export const lint = (packageJsonData: PackageJson, severity: Severity, config: any): LintIssue | boolean => {
   if (packageJsonData.hasOwnProperty(nodeName) && doVersContainFileUrl(packageJsonData, nodeName, config)) {
     return new LintIssue(lintId, severity, nodeName, message);
   }
 
   return true;
 };
-
-module.exports = {
-  lint,
-  ruleType,
-};

src/rules/no-file-devDependencies.js renamed to src/rules/no-file-devDependencies.ts

@@ -1,20 +1,18 @@
 const {doVersContainFileUrl} = require('../validators/dependency-audit');
-const LintIssue = require('../LintIssue');
+import {LintIssue} from '../lint-issue';
+import {RuleType} from '../types/rule-type';
+import {Severity} from '../types/severity';
+import {PackageJson} from 'type-fest';
 
 const lintId = 'no-file-devDependencies';
 const nodeName = 'devDependencies';
 const message = 'You are using devDependencies via url to local file. Please use devDependencies from npm.';
-const ruleType = 'optionalObject';
+export const ruleType = RuleType.OptionalObject;
 
-const lint = (packageJsonData, severity, config) => {
+export const lint = (packageJsonData: PackageJson, severity: Severity, config: any): LintIssue | boolean => {
   if (packageJsonData.hasOwnProperty(nodeName) && doVersContainFileUrl(packageJsonData, nodeName, config)) {
     return new LintIssue(lintId, severity, nodeName, message);
   }
 
   return true;
 };
-
-module.exports = {
-  lint,
-  ruleType,
-};