pahen/madge

[Feat] Specify config path via CLI

Open

#335 aberto em 18 de nov. de 2022

Ver no GitHub
 (1 comment) (5 reactions) (0 assignees)JavaScript (8.058 stars) (322 forks)batch import
cligood first issue

Description

There is currently no way to provide config path or detectiveOptions via CLI. Its blocked by commander even though it would work out of the box with rc

Guia do colaborador

Pilha de tecnologia
javascriptnodejs
Domain
clitooling
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Basic Node.js knowledge
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60
Direção de pesquisa
The issue requests adding a CLI option to specify a config file path. Currently, madge uses commander for CLI argument parsing, but does not accept a config path. The issue suggests using the 'rc' package to load configuration. To implement, look at the CLI entry point (e.g., bin/madge or similar) and add a new option like ' c' or ' config' using commander. Then integrate a config loading mechanism, possibly using 'rc' as suggested. Review existing configuration handling in the codebase to ensure compatibility.