Skip to content

Não estamos mais servindo arquivos minificados #63

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
mateusfccp opened this issue Apr 17, 2024 · 8 comments · Fixed by #67
Closed

Não estamos mais servindo arquivos minificados #63

mateusfccp opened this issue Apr 17, 2024 · 8 comments · Fixed by #67
Assignees
Labels
bug Something isn't working

Comments

@mateusfccp
Copy link
Contributor

Por algum motivo, no commit 9f5858f os arquivos minificados foram apagados e os arquivos .setup estão configurados para servir os arquivos não minificados.

Como o commit foi feito sem passar por processo de revisão, imagino que pode ter sido um engano (a pessoa pode ter desabilitado a minificação para debugar mais facilmente localmente e esqueceu de reverter). Não vejo motivo para servir os arquivos não minificados.

Proponho reverter essa parte do commit e voltar a servir os arquivos minificados, o que deve melhorar o tempo de carregamento da página, mesmo que marginalmente.

@mateusfccp mateusfccp added the bug Something isn't working label Apr 17, 2024
@ryukinix
Copy link
Member

Acredito que tenha sido minha culpa quando fui editar e achei mais conveniente deixar sem ser minificado. O ideal é deixar minificado apenas na master e o processo de deployment fazer a minificação automaticamente, senão ficaria propenso a acontecer esse mesmo deslize na próxima edição dos arquivos de css na branch develop.

@mateusfccp
Copy link
Contributor Author

Antigamente, a minificação acontecia em que momento? Não encontrei no repositório nenhum script que faça isso.

@mateusfccp
Copy link
Contributor Author

O ideal é deixar minificado apenas na master e o processo de deployment fazer a minificação automaticamente, senão ficaria propenso a acontecer esse mesmo deslize na próxima edição dos arquivos de css na branch develop.

Normalmente se usa source maps, justamente para resolver esse problema. Se minifica tudo, mas na hora de depurar os source maps vão restaurar o arquivo "original".

@ryukinix
Copy link
Member

ryukinix commented Apr 17, 2024 via email

@mateusfccp
Copy link
Contributor Author

Antigamente, a minificação acontecia em que momento? Não encontrei no repositório nenhum script que faça isso.

Favor responder essa pergunta.


Sobre o deploy automatizado, acho que está relacionado e pode ser resolvido junto, já que a minificação faz parte do CI.

@ryukinix
Copy link
Member

ryukinix commented Apr 17, 2024 via email

@mateusfccp
Copy link
Contributor Author

Imagino que eu não tenha me expressado bem. Ainda que o arquivo fosse minificado manualmente, alguma ferramenta era usada para fazê-lo. Eu imagino que o "manual" queira dizer que era feito na máquina local e depois copiado para o source; não creio que queira dizer que você, literalmente, minificava manualmente, removendo espaço por espaço e trocando nomes perfeitamente semânticos por nomes obscuros no seu editor de texto.

A pergunta é, unicamente, para entender se há a preferência de uso de alguma ferramenta específica no processo de minificação. Eu poderia usar qualquer coisa, mas estou respeitando o que já estava sendo feito anteriormente em vez de tentar impor minhas preferências.

@ryukinix
Copy link
Member

ryukinix commented Apr 18, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants