palantir/blueprint

Enable strictNullChecks compiler option for all packages

Open

#325 ouverte le 6 déc. 2016

Voir sur GitHub
 (9 commentaires) (0 réactions) (0 assignés)TypeScript (20 263 stars) (2 167 forks)batch import
Domain: toolinghelp wanted

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

Stack technique
typescriptreact
Domaine
frontenddeveloper experience
Type d'issue
refactor
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 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
TypeScript basicsFamiliarity with strictNullChecks
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40
Direction de recherche
Examine the datetime package's tsconfig.json to add 'strictNullChecks: true'. Then, compile the package to identify type errors, referring to related issues #322 and #274 for context. Fix errors by adding null checks or adjusting types, ensuring compatibility with existing code. No linked PRs or assignees exist, so work can be picked up independently.