nestjs / nest

A progressive Node.js framework for building efficient, scalable, and enterprise-grade server-side applications with TypeScript/JavaScript 🚀
https://nestjs.com
MIT License
66.9k stars 7.55k forks source link

Migrate tslint to eslint #3534

Closed quadgod closed 4 years ago

quadgod commented 4 years ago

Feature Request

Migrate tslint to eslint

Describe the solution you'd like

I would suggest to use a next configuration (just example)

// .eslintrc
{
  "parser": "@typescript-eslint/parser",
  "plugins": ["@typescript-eslint"],
  "rules": {
    "no-unused-expressions": ["error"],
    "@typescript-eslint/member-ordering": ["off"],
    "@typescript-eslint/interface-name-prefix": ["off"],
    "@typescript-eslint/explicit-member-accessibility": ["off"],
    "@typescript-eslint/quotes": ["error", "single", "avoid-escape"]
  },
  "extends": ["plugin:@typescript-eslint/recommended"]
}

Good to have it in nest-cli

What is the motivation / use case for changing the behavior?

image https://github.com/palantir/tslint

tslint is no longer supported is somebody working on tslint -> eslint migration?

kamilmysliwiec commented 4 years ago

Tracking here https://github.com/nestjs/typescript-starter/issues/69

lock[bot] commented 4 years ago

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.