palantir/blueprint

Enable strictNullChecks compiler option for all packages

Fermée

#325 ouverte le 6 déc. 2016

Voir sur GitHubGuide contributeur
 (9 commentaires) (0 réaction) (0 personne assignée)TypeScript (2 167 forks)batch import
Domain: toolinghelp wanted

Métriques du dépôt

Stars
 (20 263 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

I think enabling strictNullChecks compiler option for datetime package will help us detect and prevent a whole host of potential errors, such as #322. it'll also address #274 by making the typings more accurate (as explicit null Dates actually means something in the component).

Guide contributeur

Direction de recherche
Examinez le tsconfig.json du package datetime pour ajouter 'strictNullChecks: true'. Ensuite, compilez le package pour identifier les erreurs de type, en vous référant aux issues connexes #322 et #274 pour le contexte. Corrigez les erreurs en ajoutant des vérifications de null ou en ajustant les types, en veillant à la compatibilité avec le code existant. Il n'y a pas de PR liés ni de responsables assignés, le travail peut donc être entrepris indépendamment.
Stack technique
typescriptreact
Domaine
frontend
Type d'issue
Maintenance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55