Terminal: ColorTool: correções de bugs do analisador e refatoramento de código pequeno

Criado em 21 abr. 2019  ·  3Comentários  ·  Fonte: microsoft/terminal

Estou usando o ColorTool como parte do gerenciamento de um tema para cmd / powershell e encontrei alguns problemas ao usá-lo:

  1. Mensagens de erro sendo lançadas de analisadores que não são usados, por exemplo, o analisador INI tenta executar mesmo se você estiver usando itermcolors , então ele lança erros porque XML não é um arquivo INI válido.
  2. Índices de primeiro plano / plano de fundo não são salvos ao usar --defaults (ou --both ).

Essas são correções fáceis e fico feliz em corrigi-las. Eu estava pensando em limpar um pouco o código enquanto estou lá, a saber:

  1. Permitindo que os analisadores sejam executados condicionalmente, portanto, nem todos os analisadores são executados para todos os arquivos (deve silenciar alguns erros)
  2. Adicionando uma interface comum para "definir as propriedades atuais do console" e "padrões do console", para que seja mais difícil esquecer a implementação da funcionalidade de ambos.

No entanto, antes de começar a trabalhar nisso, gostaria de avisá-lo e ter certeza de que não estava trabalhando em oposição a alguém no projeto. Você estaria interessado em um PR que corrigisse os 2 pontos acima?

Obrigado!

Area-Settings Issue-Bug Product-Colortool Resolution-Fix-Available

Todos 3 comentários

@waf , seria uma mudança muito bem-vinda. Ninguém do nosso lado está trabalhando em nenhum deles.

@ DHowett-MSFT obrigado, abri o número 418 para isso.

Acredito que isso já esteja disponível no lançamento que fiz algumas semanas atrás.

Esta página foi útil?
0 / 5 - 0 avaliações