Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

@deprecated should be associated to a specific overload. #2880

Closed
vilicvane opened this issue Jun 4, 2017 · 2 comments · Fixed by #2883
Closed

@deprecated should be associated to a specific overload. #2880

vilicvane opened this issue Jun 4, 2017 · 2 comments · Fixed by #2883

Comments

@vilicvane
Copy link
Contributor

Bug Report

  • TSLint version: 5.4.2
  • TypeScript version: 2.3.4
  • Running TSLint via: CLI, VSCode

TypeScript code being linted

function foo<T>(): T;
/** @deprecated */
function foo(bar: any): any;
function foo() { }

foo<number>();

with tslint.json configuration:

{
  "extends": "vts"
}

Actual behavior

foo is deprecated. (deprecation)

Expected behavior

No error.

@buu700
Copy link
Contributor

buu700 commented Aug 14, 2017

Running into this with Angular Injector: https://github.com/angular/angular/blob/d9d00bd/packages/core/src/di/injector.ts#L53

@ajafff
Copy link
Contributor

ajafff commented Aug 14, 2017

For the reference: I have the same problem as @buu700

#2883 will fix this. It's only waiting for a reviewer.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants