Microsoft/TypeScript

Suggestion: Change `Diagnostic.file` to be `SourceFileLike` instead of `SourceFile`.

Ouverte

#18 013 ouverte le 24 août 2017

Voir sur GitHubGuide contributeur
 (1 commentaire) (1 réaction) (0 personne assignée)TypeScript (13 395 forks)batch import
Domain: APIHelp WantedSuggestion

Métriques du dépôt

Stars
 (108 860 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Diagnostic.file is of type SourceFile which forces plugins to cast a non-SourceFile to a SourceFile to report diagnostic messages and guess which parts of SourceFile are not used by tsserver.

Consider changing file to be something like SourceFileLIke or SourceMapSource that specifies the exact requirements that must be supplied by plugins for this field.

Guide contributeur

Direction de recherche
Explorez le code source du compilateur TypeScript pour comprendre l'utilisation actuelle de `Diagnostic.file` et identifier les propriétés minimales requises pour les plugins. Proposez une nouvelle interface comme `SourceFileLike` basée sur l'utilisation réelle.
Stack technique
typescript
Domaine
backendtooling
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScriptTypeScript compiler internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
20