rescript-lang/rescript-vscode

parseCompilerLogOutput crashes when first error is graphql deprecation warning

Ouverte

#386 ouverte le 7 avr. 2022

Voir sur GitHubGuide contributeur
 (12 commentaires) (0 réaction) (1 personne assignée)ReScript (63 forks)github user discovery
buggood first issuehelp wanted

Métriques du dépôt

Stars
 (350 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Hi Folks. Found a case here. given a codebase using rescript-apollo, with a deprecated field warning, the parser will fail its "File " branch based on the next line not being Warning, and end up in its last case where it attempts to push a follow up line to an empty parsedDiagnostics array.

I can see some pushback on the apollo output here maybe, but would it be better to make the "File " branch more permissive, or allow a new parsed diagnostic to be created there in that non-empty-line case?

Thanks Alex

File "/Users/alexmouton/Documents/code/noble/web/src/packs/reactor/PackCapabilityRunSweepUpdateHf.res", line 45, characters 8-26:
45 | ........entDataRecordId
45 |   .....
Warning 22 [preprocessor]: Field "parentDataRecordId" has been deprecated. Reason: Prefer parent_data_record
File "/Users/alexmouton/Documents/code/noble/web/src/packs/reactor/PackCapabilityRunSweepUpdateHf.res", line 42, characters 40-53:
42 | ........................................ectedInput
42 |   .......

Guide contributeur

Direction de recherche
Examinez la fonction parseCompilerLogOutput pour gerer les avertissements de deprecation commencant par 'Warning' apres une ligne 'File', en garantissant qu'elle ne plante pas sur un tableau parsedDiagnostics vide.
Stack technique
typescriptjavascript
Domaine
developer experiencetooling
Type d'issue
Bug
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-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitVSCode
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60