Remove tslint comments

Author: RichDom2185

src/__tests__/block-scoping.test.ts

@@ -1,7 +1,7 @@
 import { test } from 'vitest'
 import { Chapter } from '../langs'
 import { stripIndent } from '../utils/formatters'
-import { expectParsedError, expectFinishedResult } from '../utils/testing'
+import { expectFinishedResult, expectParsedError } from '../utils/testing'
 
 // This is bad practice. Don't do this!
 test('standalone block statements', () => {
@@ -145,7 +145,6 @@ test('Error when accessing temporal dead zone', () => {
     `).toEqual("Line 3: ReferenceError: Cannot access 'a' before initialization")
 }, 30000)
 
-// tslint:disable-next-line:max-line-length
 test('In a block, every going-to-be-defined variable in the block cannot be accessed until it has been defined in the block.', () => {
   return expectParsedError(stripIndent`
       const a = 1;

src/__tests__/stringify.test.ts

@@ -97,7 +97,6 @@ test('Correctly handles circular structures with multiple entry points', async (
 })
 
 // The interpreter runs into a MaximumStackLimitExceeded error on 1000, so reduced it to 100.
-// tslint:disable:max-line-length
 test('String representation of huge lists are nice', async () => {
   const {
     result: { value }
@@ -194,7 +193,6 @@ test('String representation of huge lists are nice', async () => {
             [89, [90, [91, [92, [93, [94, [95, [96, [97, [98, [99, [100, null]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]"
           `)
 })
-// tslint:enable:max-line-length
 
 test('String representation of huge arrays are nice', async () => {
   const {
@@ -432,7 +430,6 @@ test('String representation of undefined is nice', () => {
   return expectFinishedResult(`stringify(undefined);`).toEqual('undefined')
 })
 
-// tslint:disable:max-line-length
 test('String representation with no indent', async () => {
   const {
     result: { value }

src/createContext.ts

@@ -7,6 +7,7 @@ import { call_with_current_continuation } from './cse-machine/continuations'
 import Heap from './cse-machine/heap'
 import { Transformers } from './cse-machine/interpreter'
 import { cset_apply, cset_eval } from './cse-machine/scheme-macros'
+import { Chapter, Variant, type LanguageOptions } from './langs'
 import * as list from './stdlib/list'
 import { list_to_vector } from './stdlib/list'
 import { listPrelude } from './stdlib/list.prelude'
@@ -25,7 +26,6 @@ import * as stream from './stdlib/stream'
 import { streamPrelude } from './stdlib/stream.prelude'
 import { createTypeEnvironment, tForAll, tVar } from './typeChecker/utils'
 import type { Context, CustomBuiltIns, Environment, NativeStorage, Value } from './types'
-import { Chapter, Variant, type LanguageOptions } from './langs'
 import * as operators from './utils/operators'
 import { stringify } from './utils/stringify'
 
@@ -366,7 +366,6 @@ export const importBuiltins = (context: Context, externalBuiltIns: CustomBuiltIn
     defineBuiltin(
       context,
       'apply_in_underlying_javascript(fun, args)',
-      // tslint:disable-next-line:ban-types
       (fun: Function, args: Value) => fun.apply(fun, list_to_vector(args))
     )
 
@@ -389,7 +388,6 @@ export const importBuiltins = (context: Context, externalBuiltIns: CustomBuiltIn
     defineBuiltin(context, 'is_NaN(val)', misc.is_NaN)
     defineBuiltin(context, 'has_own_property(obj, prop)', misc.has_own_property)
     defineBuiltin(context, 'alert(val)', alert)
-    // tslint:disable-next-line:ban-types
     defineBuiltin(context, 'timed(fun)', (f: Function) =>
       misc.timed(context, f, context.externalContext, externalBuiltIns.rawDisplay)
     )

src/cse-machine/interpreter.ts

@@ -5,23 +5,23 @@
  * and the legacy interpreter
  */
 
-/* tslint:disable:max-classes-per-file */
 import type es from 'estree'
 import { isArray } from 'lodash'
 
 import type { IOptions } from '..'
+import { isSchemeLanguage } from '../alt-langs/mapper'
 import { UNKNOWN_LOCATION } from '../constants'
 import * as errors from '../errors/errors'
 import { RuntimeSourceError } from '../errors/runtimeSourceError'
 import { checkEditorBreakpoints } from '../stdlib/inspector'
 import type {
   Context,
   ContiguousArrayElements,
+  Node,
+  NodeTypeToNode,
   Result,
-  Value,
   StatementSequence,
-  Node,
-  NodeTypeToNode
+  Value
 } from '../types'
 import * as ast from '../utils/ast/astCreator'
 import {
@@ -34,14 +34,16 @@ import { evaluateBinaryExpression, evaluateUnaryExpression } from '../utils/oper
 import * as rttc from '../utils/rttc'
 import * as seq from '../utils/statementSeqTransform'
 import { checkProgramForUndefinedVariables } from '../validator/validator'
-import { isSchemeLanguage } from '../alt-langs/mapper'
 import Closure from './closure'
 import {
   Continuation,
   isCallWithCurrentContinuation,
   makeDummyContCallExpression
 } from './continuations'
 import * as instr from './instrCreator'
+import { flattenList, isList } from './macro-utils'
+import { Transformer } from './patterns'
+import { isApply, isEval, schemeEval } from './scheme-macros'
 import { Stack } from './stack'
 import {
   type AppInstr,
@@ -83,9 +85,6 @@ import {
   setVariable,
   valueProducing
 } from './utils'
-import { isApply, isEval, schemeEval } from './scheme-macros'
-import { Transformer } from './patterns'
-import { flattenList, isList } from './macro-utils'
 
 /**
  * The control is a list of commands that still needs to be executed by the machine.

src/editors/ace/modes/source.ts

@@ -1,16 +1,14 @@
 import { Variant } from '../../../langs'
 import { SourceDocumentation } from '../docTooltip'
 
-/* tslint:disable */
-
 /**
  * Source Mode for Ace Editor
  * (Modified from javascript mode in default brace package)
  * The link to the original JavaScript mode can be found here:
  * https://github.com/ajaxorg/ace-builds/blob/master/src/mode-javascript.js
  *
  * Changes includes:
- * 1) change code styles so that it passes tslint test
+ * 1) change code styles
  * 2) refactor some code to ES2015 class syntax
  * 3) Encapsulate the orginal mode and higlightrules in two selectors so as to change according to source chapter
  * 4) changed regex to mark certain operators in pink

src/errors/errors.ts

@@ -1,12 +1,10 @@
-/* tslint:disable: max-classes-per-file */
-/* tslint:disable:max-line-length */
 import { baseGenerator, generate } from 'astring'
 import type es from 'estree'
 
 import { UNKNOWN_LOCATION } from '../constants'
 import type { Node, Value } from '../types'
 import { stringify } from '../utils/stringify'
-import { ErrorType, ErrorSeverity, type SourceError } from './base'
+import { ErrorSeverity, ErrorType, type SourceError } from './base'
 import { RuntimeSourceError } from './runtimeSourceError'
 
 //Wrap build-in function error in SourceError

src/errors/timeoutErrors.ts

@@ -1,4 +1,3 @@
-/* tslint:disable:max-classes-per-file */
 import { JSSLANG_PROPERTIES } from '../constants'
 import type { Node } from '../types'
 import { stripIndent } from '../utils/formatters'

src/errors/typeErrors.ts

@@ -1,15 +1,12 @@
 import { generate } from 'astring'
 import type es from 'estree'
-
 import { UNKNOWN_LOCATION } from '../constants'
 import type * as tsEs from '../typeChecker/tsESTree'
 import type { Node, NodeWithInferredType, SArray, Type } from '../types'
+import { getSourceVariableDeclaration } from '../utils/ast/helpers'
 import { simplify, stripIndent } from '../utils/formatters'
 import { typeToString } from '../utils/stringify'
-import { getSourceVariableDeclaration } from '../utils/ast/helpers'
-import { ErrorType, ErrorSeverity, type SourceError } from './base'
-
-// tslint:disable:max-classes-per-file
+import { ErrorSeverity, ErrorType, type SourceError } from './base'
 
 export class InvalidArrayIndexType implements SourceError {
   public type = ErrorType.TYPE

src/name-extractor/index.ts

@@ -8,9 +8,9 @@ import { memoizedGetModuleDocsAsync, memoizedGetModuleManifestAsync } from '../m
 import type { ModuleDocsEntry } from '../modules/moduleTypes'
 import { isSourceModule } from '../modules/utils'
 import syntaxBlacklist from '../parser/source/syntax'
+import type { Context, Node } from '../types'
 import { getImportedName, getModuleDeclarationSource } from '../utils/ast/helpers'
 import { isDeclaration, isImportDeclaration, isNamespaceSpecifier } from '../utils/ast/typeGuards'
-import type { Context, Node } from '../types'
 
 export enum DeclarationKind {
   KIND_IMPORT = 'import',
@@ -173,7 +173,6 @@ export async function getProgramNames(
 
   while (queue.length > 0) {
     // Workaround due to minification problem
-    // tslint:disable-next-line
     const node = queue.shift()!
     if (isFunction(node)) {
       // This is the only time we want raw identifiers

src/stdlib/index.ts

@@ -1,6 +1,6 @@
 import createContext from '../createContext'
-import type { Value } from '../types'
 import { Chapter } from '../langs'
+import type { Value } from '../types'
 import * as list from './list'
 import * as misc from './misc'
 import * as parser from './parser'
@@ -50,7 +50,6 @@ export const chapter_4 = {
   ...chapter_3,
   parse: (str: string, chapter: Chapter) => parser.parse(str, createContext(chapter)),
   tokenize: (str: string, chapter: Chapter) => parser.tokenize(str, createContext(chapter)),
-  // tslint:disable-next-line:ban-types
   apply_in_underlying_javascript: (fun: Function, args: Value) =>
     fun.apply(fun, list.list_to_vector(args))
 }
@@ -61,7 +60,6 @@ export const chapter_library_parser = {
   is_NaN: misc.is_NaN,
   has_own_property: misc.has_own_property
   // defineBuiltin(context, 'alert(val)', alert)
-  // tslint:disable-next-line:ban-types
   // timed: (f: Function: context: Context) => misc.timed(context, f, context.externalContext, externalBuiltIns.rawDisplay),
 }