lirantal/detect-secrets

Enhancement: Support for enabling debug logs via flags

Fermée

#1 ouverte le 5 juil. 2019

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)JavaScript (5 forks)github user discovery
good first issuestale

Métriques du dépôt

Stars
 (52 étoiles)
Métriques de merge PR
 (Merge moyen 6j 13h) (4 PRs mergées en 30 j)

Description

Is your feature request related to a problem? Please describe. If I'm trying to debug how detect-secrets work then I need to run it with DEBUG=detect-secrets which lint-staged isn't friendly with. Instead, I'd want to pass it some flags like --debug in order to enable debugging.

Describe the solution you'd like

  1. Add a --debug flag to enable debugging
  2. This should enable debug dynamically via debug.enable('detect-secrets')
  3. Discard the flag from process.argv so it doesn't get passed to the actual detect-secrets-hook tool

Describe alternatives you've considered Open to other ideas

Guide contributeur

Direction de recherche
Examinez la base de code pour trouver où les arguments de la ligne de commande sont analysés (par exemple avec yargs ou process.argv). Implémentez ensuite un drapeau ` debug` qui active l'espace de noms de débogage 'detect secrets' via le package `debug`, et assurez vous que le drapeau est supprimé avant de passer les arguments à l'outil sous jacent.
Stack technique
javascriptnodejs
Domaine
cli
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.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70