anuraghazra/github-readme-stats

Migrate codebase to Typescript

Open

#2 052 ouverte le 25 sept. 2022

Voir sur GitHub
 (35 commentaires) (2 réactions) (3 assignés)JavaScript (33 282 forks)batch import
enhancementhacktoberfesthelp wanted

Métriques du dépôt

Stars
 (79 424 stars)
Métriques de merge PR
 (Merge moyen 21j 13h) (1 PR mergée en 30 j)

Description

Is your feature request related to a problem? Please describe.

We currently use javascript + type definitions. I, however, think migrating the project to typescript will significantly improve the development experience since it allows us to catch errors earlier.

Extra information

There were some attempts in the past that people can build upon if they want to take on this issue:

Additionally, @HwangTaehyun translated part of the codebase to typescript when creating https://github.com/HwangTaehyun/github-contributor-stats.

Guide contributeur

Direction de recherche
Examinez les tentatives de migration précédentes dans les PR #1288 et #832, ainsi que le fork de HwangTaehyun (github contributor stats). Les définitions de types existantes dans src/fetchers/types.d.ts doivent être exploitées. La migration implique de convertir tous les fichiers .js en .ts, de mettre à jour les imports et d'assurer la sécurité des types dans l'ensemble de la base de code. Étudiez les commentaires et le diff des PR liées pour comprendre les blocages et les progrès réalisés.
Stack technique
javascripttypescript
Domaine
backend
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.
3-5 jours
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.
Plutôt claire
Prérequis
TypeScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40