Skip to content

Fix lint tooling and move to Node 22 #124

Description

@CatarinaGamboa

Why: npm test runs lint first (pretest), and ESLint 7 with @typescript-eslint 4 fails on every file with Parsing error: DeprecationError: 'originalKeywordKind' has been deprecated. No test can run until this is fixed. The current Vitest and @vscode/test-electron releases need Node 22, and CI uses Node 20.

What:

  • Move to ESLint 9+ with a flat eslint.config.mjs and typescript-eslint 8.
  • Take lint out of pretest; it becomes its own CI step.
  • Remove unused devDependencies (ftp, rimraf, mkdirp, get-port, old vsce), and use @vscode/vsce.
  • Use Node 22 in publish.yml.
  • Remove the test script that points at the missing out/test/runTest.js.

Done when: npm run lint passes locally.

References:

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    testingTesting related

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions