microsoft / microsoft/TypeScript
Quick info shouldn't default to showing jsdoc from first overload
Ouverte
@sandersn y travaille déjà.
Depuis le 12/8/2021.
Needs Proposal
Suggestion
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.4k
- Merge moyen
- 1 j 19 h
- PR mergées (30 j)
- 117
Description
https://github.com/microsoft/TypeScript/issues/43053
This is still happening at least with the tap function from rxjs. VSCode 1.58.2. TS version in project: 4.3.5
Deprecation warning shows up regardless of which overloaded signature you use. Should not show up for the last two signatures.
import { interval } from 'rxjs';
import { tap } from 'rxjs/operators';
interval(5000)
.pipe(
tap(console.log), // "deprecated - Use an observer instead of a complete callback"
tap({ next: console.log, error: console.log, complete: console.log }) // "deprecated
)
rxjs/tap.d.ts:
import { MonoTypeOperatorFunction, PartialObserver } from '../types';
/** @deprecated Use an observer instead of a complete callback */
export declare function tap<T>(next: null | undefined, error: null | undefined, complete: () => void): MonoTypeOperatorFunction<T>;
/** @deprecated Use an observer instead of an error callback */
export declare function tap<T>(next: null | undefined, error: (error: any) => void, complete?: () => void): MonoTypeOperatorFunction<T>;
/** @deprecated Use an observer instead of a complete callback */
export declare function tap<T>(next: (value: T) => void, error: null | undefined, complete: () => void): MonoTypeOperatorFunction<T>;
export declare function tap<T>(next?: (x: T) => void, error?: (e: any) => void, complete?: () => void): MonoTypeOperatorFunction<T>;
export declare function tap<T>(observer: PartialObserver<T>): MonoTypeOperatorFunction<T>;
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Évaluation
Cette issue n'a pas encore été évaluée.