codeclimate/codeclimate-duplication

Please support Typescript

trashhalo opened this issue Β· 7 comments

would really love to have this feature implemented in CodeClimate

Hey @trashhalo @XMichaelMichaelX,

We now have a beta Typescript engine available, maintained by @tkqubo!

https://github.com/tkqubo/codeclimate-tslint

You can use it on https://codeclimate.com or via the CLI with the following addition in your .codeclimate.yml configuration file:

engines:
  tslint:
    enabled: true
    channel: beta

Edit: I realize the tslint engine in-progress doesn't support duplication analysis. We would welcome a PR to this engine for Typescript support or a standalone engine addressing Typescript duplication analysis.

Just some feedback, but the duplication analysis seems... somewhat dubious:

image

Anything I may have misconfigured?

This is supported. Please close. Open new tickets for separate issues so they don't get buried.

@zenspider I'm having hard time finding documentation on TypeScript support. May I kindly ask you to point to the docs before closing the issue?

Sorry. I'm just a contractor for codeclimate. I don't actually know where any of the doco is. I just fix stuff. πŸ˜›

That said, I did just fix some of the stuff around both javascript and typescript. If you add it to your enabled languages, it should Just Workβ„’. Example:

engines:
  duplication:
    enabled: true
    config:
      languages:
      - javascript
      - typescript
Lorel commented

Indeed, duplication does support Typescript:

"typescript" => ::CC::Engine::Analyzers::TypeScript::Main,

So this issue should be closed

But actually, I didn't find out any mention of all the supported languages here
Duplication is a very great tool, we should mention this clearly to encourage people of using it, this one worth it