Conversation
gullitmiranda
left a comment
There was a problem hiding this comment.
@igraguiar esse PR ainda está como Draft, mas vou deixar o review para podermos finalizar isso.
Os pontos que precisa ser visto são:
-
Atualizar o
@actions/coreque é o motivo do ter deixado de funcionar (https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/) -
O artefato expira em 90 dias, sendo assim precisamos encontrar uma alternativa, as sugestões seriam
a. Mais simples: adicionar um schedule para o workflow test.yml para ser executado com uma frequência menor que 90 dias
b. Ideal: Fazer o upload do artefato para releases.
Sobre o 2.b, se eu não estou enganado o problema era com o arquivo compactado no release. Ao extrair ele perdia as referências necessárias
- se não for muito trabalho, já fazer a atualização das dependências vulneráveis: https://github.com/yubeio/setup-imagemagick/security/dependabot
gullitmiranda
left a comment
There was a problem hiding this comment.
@igraguiar levantei um ponto nos comentários.
Além disso, com as mudanças no PR https://github.com/yubeio/hive/pull/424 isso não vai ser mais necessário.
Utilizar o ghostscript resolve tanto a questão de precisar do imagemagick compilado a partir do source quanto as miniaturas estarem sendo gerada erradas.
| # Required, uploaded artifact name | ||
| name: ImageMagick-${{ env.IMAGEMAGICK_VERSION }}-x64-precompiled | ||
| # Optional, directory where to extract artifact | ||
| path: ../ImageMagick-${{ env.IMAGEMAGICK_VERSION }}-x64-precompiled |
There was a problem hiding this comment.
@igraguiar será que faz sentido ativar isso para o workflow final?
pq se isso tiver ativado o build pode não acontecer como esperado, já que ele pode usar o cache.
A ideia de ter isso comentado era para poder pular o processo de compilação enquanto estivesse trabalhando nos outros steps.
There was a problem hiding this comment.
Eu pensei em usar pois se a versão desejada não mudar, em teoria não haveria problema em usar a versão do cache. Já que também temos a etapa que testa se o artefato funciona conforme o desejado. @gullitmiranda Nós vamos continuar com a correção desse projeto? Como você mencionou no slack que talvez não precisaríamos e vendo as alterações no ci/cd do Hive acabei não retomando aqui.
There was a problem hiding this comment.
Eu pensei em usar pois se a versão desejada não mudar, em teoria não haveria problema em usar a versão do cache
vc está certo @igraguiar, faz bastante sentido
@gullitmiranda Nós vamos continuar com a correção desse projeto? Como você mencionou no slack que talvez não precisaríamos e vendo as alterações no ci/cd do Hive acabei não retomando aqui.
Não precisamos mais @igraguiar. pode deixar isso para lá. Se tivesse pronto, valeria a pena fazer merge.
Mas como não está, não faz mais sentido investir nisso agora
Recompiled main.ts to fix code dependencies.