File extension validation

rhandrade / tray-theme

CLI criado para ajudar desenvolvedores a criarem ótimos temas para Tray.

MIT License

20 stars 3 forks source link

File extension validation #16

Closed leolopes closed 3 years ago

leolopes commented 3 years ago

Relacionado a esta issue: https://github.com/rhandrade/tray-theme/issues/10 Não considero isso pronto, mas sim uma oportunidade para avaliar os caminhos abordados.

O que fiz aqui?

Abstraí uma função para verificar se o arquivo pode ser subido ou não. Hoje, apenas verifica a extensão do arquivo, mas esta função poderá ser melhorada para ignorar padrões de pasta (como a node_modules, que hoje ele tenta subir). Editado: já melhorei bastante a função, incluindo ignorar certas pastas.
Abstraí uma função para preparar os parâmetros para envio de arquivos pela API
Usei essas funções no watch mas também no upload, padronizando ambos.

Gostaria que avaliassem aqui para ver se devo seguir no caminho. Se quiserem já fazer merge, também já podem se beneficiar dessa pequena melhoria.

thiagofloriano commented 3 years ago

Se a proposta aqui é só validar extensão de arquivos, creio que a lógica para ignorar os node_modules ou outras pastas de ambiente dev seja interessante separar em outra issue.

Talvez valha até criar parâmetros para o usuário poder ignorar pastas ou extensões selecionadas.

leolopes commented 3 years ago

Se a proposta aqui é só validar extensão de arquivos, creio que a lógica para ignorar os node_modules ou outras pastas de ambiente dev seja interessante separar em outra issue.

Acabei já tratando dessa questão no último commit.

leolopes commented 3 years ago

@rhandrade, acabo de fazer as abstrações necessárias. Tudo ainda funciona, mas vale o teste.

rhandrade commented 3 years ago

Show @leolopes.... Vou fazer mais alguns testes e tudo dando ok eu aprovo ela assim que começarmos a trabalhar com a versão 1.1.0.

Acredito que finalizando os testes sobre os sistemas operacionais já conseguimos lançar uma versão release candidate e por fim a versão estável 1.0.0.

Muito obrigado pela ajuda até agora. Está sendo incrível!

leolopes commented 3 years ago

@rhandrade, fico feliz em poder ajudar. Essa ferramenta foi crucial para os novos funcionários, que não conseguem de jeito nenhum instalar o CLI do opencode 😞

Vamos esperar então os testes do @thiagofloriano , no MacOS e no Ubuntu 20.

Como falei na issue, eu tenho o Ubuntu 18.

rhandrade commented 3 years ago

Eu entendo bem @leolopes... Foi exatamente pelo mesmo motivo que desenvolvi ele. Pode contar comigo pro que precisar, e desculpa a demora... esses dias estão bem corridos.

leolopes commented 3 years ago

Ótimo, aguardando ansioso este merge! Edit: notei que fizeste um comentário na issue. Depois vou alterar aqui então para aceitar SCSS.

leolopes commented 3 years ago

@rhandrade, pronto para o merge.