honkit/honkit

Replace bash-color with https://github.com/lukeed/kleur

Open

#272 ouverte le 21 juin 2022

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)TypeScript (3 450 stars) (259 forks)user submission
Type: Refactoringgood first issue

Description

https://github.com/mbilokonsky/bash-color does not support NO_COLOR=1 We want to get NO_COLOR supports by replacing bashcolor with another package like https://github.com/lukeed/kleur

Guide contributeur

Stack technique
typescriptnodejs
Domaine
clibackend
Type d'issue
feature
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 hours
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
TypeScriptNode.jsunderstanding of NO COLOR
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
This issue involves replacing the `bash color` dependency with `kleur` in the file `packages/honkit/src/utils/logger.ts` to add support for the `NO COLOR=1` environment variable. The change should ensure that the new kleur API is used correctly and that existing color functionality is preserved. Checking the current usage of `bash color` in that file will help understand what needs to be replaced. Also, reviewing the kleur documentation for its API and how it handles `NO COLOR` is necessary.