sindresorhus/eslint-plugin-unicorn

Improve DX around `prefer-global-this` in TypeScript files

Fermée

#2 488 ouverte le 25 oct. 2024

Voir sur GitHubGuide contributeur
 (0 commentaire) (1 réaction) (0 personne assignée)JavaScript (468 forks)user submission
docsenhancementhelp wanted

Métriques du dépôt

Stars
 (5 022 étoiles)
Métriques de merge PR
 (Merge moyen 4h 30m) (26 PRs mergées en 30 j)

Description

A lot of types in the wild are either only set on Window or differ between window.* and globalThis.*. These types are wrong, but they do come up when this rule is applied.

There are a few ways that this rule can help with the migration:

  • disable the rule's auto-fix in TS files
  • extend the message to specify how to specify the type
  • suggest solutions in the rule's readme

Examples:

Originally posted by @fregante in https://github.com/sindresorhus/eslint-plugin-unicorn/issues/2482#issuecomment-2424797467

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle de la règle `prefer global this`, identifiez comment les fichiers TypeScript sont traités et envisagez de mettre en œuvre une ou plusieurs des améliorations suggérées : désactiver la correction automatique dans les fichiers TS, étendre le message d'erreur ou ajouter des exemples dans la documentation.
Stack technique
typescriptjavascript
Domaine
developer experiencetooling
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.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ESLintTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55