microsoft / microsoft/TypeScript
[Typescript] Give a visual indicator when overriding methods with the `@sealed` js/tsdoc annotation
Personne n'a encore pris cette issue.
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.3k
- Merge moyen
- 2 j 4 h
- PR mergées (30 j)
- 132
Description
I notice that vscode has builtin support for reading the @deprecated jsdoc / tsdoc annotation that can appear on methods and variables, which causes them to appear with a strikethrough.
It would be useful if there was a similar visual indicator when a subclass attempts to override a method that is marked as @sealed in its superclass, or when it tries to inherit from a sealed superclass. (not a strikethrough, but perhaps squiggles, or any kind of feedback).
Given that there is already machinery that looks through the class hierarchy to determine that a field defined in a parent class is marked as deprecated, I would presume it wouldn't be that difficult to also identify the @sealed annotation.
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.
Piste de recherche
Commencez par localiser la gestion existante des annotations @deprecated et la recherche dans la hiérarchie de classes mentionnée dans l’issue. Déterminez comment @sealed est représenté et où les retours concernant la surcharge ou l’héritage sont produits ; le travail est terminé lorsqu’une sous-classe qui surcharge une méthode scellée ou hérite d’une superclasse scellée reçoit un diagnostic ou un indicateur visible.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- compilers
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100