typed-ember / ember-cli-tslint Goto Github PK

Hey just let you know I've submitted another issue here: kratiahuja/broccoli-tslinter#25
But do you have any plan to switch the dependency of broccoli-tslinter? Or is possible to contact that author. It's not being updated for 5 months.

Currently, rule violations do not state the name of the rule in question being violated.

User Story:

As a developer, I need to be able to know the name of the rule being violated in order to decide whether I want to disable the rule or alter my code to align with it.

There is an error with this repository's Renovate configuration that needs to be fixed. As a precaution, Renovate will stop PRs until it is resolved.

Error type: Cannot find preset's package (@)

In order to avoid repeated Using tslint.json as the default file for linting rules messages, we need to pass a configuration option to broccoli-tslinter.

Being able to set other options would likely to useful too.

Currently an empty options hash is being passed to TSLint().

Our project, like many TypeScript projects, is in a slow process of transition. We want to be linting both our unconverted JavaScript and our converted and new TypeScript.

In our specific case, we also have separate eslint and tslint configs, so we want both to run!

Currently if you create a new ember app, add ember-cli-typescript and ember-cli-tslint, followed by something like

ember g route home

and then define a route as shown below, you'll get two warnings from the TypeScript compiler due to ember-cli-typescript using the --noUnusedParameters flag

import Route from "@ember/routing/route";
import { Ember } from "ember";
import Model from "ember-data/model";

export default class Home extends Route.extend({
  // anything which *must* be merged to prototype here
}) {
  // normal class body definition here
  public async afterModel(
    resolvedModel: Model,            // WARNING: 'resolvedModel' is declared but its value is never read.
    transition: Ember.Transition     // WARNING: 'transition' is declared but its value is never read.
   ) {}
}

The TS team has adopted leading underscores as way of treating unused variables without losing developer intent (i.e., naming a parameter _ doesn't leave info for fellow developers who may want to use it in the future -- _model gives them a clue).

Unfortunately, in an ember app, this leads to a tslint error. Apparently Palantir feels that unused parameters of any kind should be treated as errors.

https://github.com/palantir/tslint/blob/9d7db479596c50bbf89fa87d498ab927763a6eec/src/configs/recommended.ts#L187-L192

import Route from "@ember/routing/route";
import { Ember } from "ember";
import Model from "ember-data/model";

export default class Home extends Route.extend({
  // anything which *must* be merged to prototype here
}) {
  // normal class body definition here
  public async afterModel(
    _resolvedModel: Model,            // ERROR: [tslint] variable name must be in lowerCamelCase, PascalCase or UPPER_CASE (variable-name)
    _transition: Ember.Transition     // ERROR: [tslint] variable name must be in lowerCamelCase, PascalCase or UPPER_CASE (variable-name)
   ) {}
}

This leaves developers with guaranteed errors or warnings by default, the first time they run into a situation where they must disregard a function parameter.

Let's adopt Microsoft's convention and allow these leading underscores by default across all ember apps. We'd need to allow this by default for JS and TS.

    "variable-name": {
      "options": ["allow-leading-underscore"]
    }

Hi, thanks for the great plugin!

When I'm using SonarTS I get a bunch of:

Warning: The 'no-array-delete' rule requires type information.
Warning: The 'no-collection-size-mischeck' rule requires type information.
...

In the README it says to pass something like --project ./tsconfig.json to tslint. Is this possible when using ember-cli-tslint?