Skip to content

Migrate from tslint to oxlint#344

Open
Copilot wants to merge 2 commits into
masterfrom
copilot/migrate-from-tslint-to-oxlint
Open

Migrate from tslint to oxlint#344
Copilot wants to merge 2 commits into
masterfrom
copilot/migrate-from-tslint-to-oxlint

Conversation

Copilot AI commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Replaces the deprecated tslint linter with oxlint, using a TypeScript config file as requested.

Changes

  • oxlint.config.ts — new TypeScript config using defineConfig from oxlint; enables typescript plugin, sets browser env, and disables eqeqeq/no-empty/no-bitwise/no-console to preserve the previous tslint rule set
  • package.json — swaps tslint for oxlint ^1.70.0; adds "type": "module" to support loading the .ts config without a Node.js warning
  • Makefile — updates lint target to oxlint -c oxlint.config.ts ts/
  • tslint.json — deleted
import { defineConfig } from "oxlint";

export default defineConfig({
	plugins: ["typescript"],
	env: { browser: true },
	rules: {
		"eqeqeq": "off",
		"no-empty": "off",
		"no-bitwise": "off",
		"no-console": "off",
	},
});

Copilot AI changed the title [WIP] Migrate from tslint to oxlint with ts config Migrate from tslint to oxlint Jun 17, 2026
Copilot AI requested a review from donatj June 17, 2026 14:24
@donatj donatj marked this pull request as ready for review June 17, 2026 14:27
Copilot AI review requested due to automatic review settings June 17, 2026 14:27

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Migrates the project’s TypeScript linting from deprecated TSLint to Oxlint, updating the repo’s Node tooling configuration so make lint uses the new linter.

Changes:

  • Remove tslint.json and replace TSLint dependency with oxlint.
  • Add oxlint.config.ts and update the lint Makefile target to run oxlint against ts/.
  • Adjust package.json to support the new lint configuration and update package-lock.json accordingly.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tslint.json Removed legacy TSLint configuration.
package.json Adds ESM package mode and swaps TSLint for Oxlint dependency.
package-lock.json Updates lockfile to reflect removal of TSLint and addition of Oxlint + platform bindings.
oxlint.config.ts Introduces Oxlint configuration (TypeScript plugin + selected rule disables).
Makefile Updates lint target to invoke oxlint with the new config.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json
Comment on lines 3 to 5
"description": "The App",
"type": "module",
"repository": {
Comment thread package.json
{
"name": "shieldeddotdev",
"description": "The App",
"type": "module",
Comment thread oxlint.config.ts
browser: true,
},
rules: {
// Disabled to maintain parity with the previous tslint configuration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants