Skip to content
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

feat(check the installed docker version): script to check version #1810

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rodrigoKulb
Copy link
Contributor

Script to check the installed docker and compose dependency

Mudanças realizadas

Aqui vai uma sugestão para verificar a dependência do docker, a ideia partiu do inssue #1808

Tipo de mudança

[x] Nova funcionalidade

Checklist:

  • As modificações não geram novos logs de erro ou aviso (warning).
  • Eu adicionei testes que provam que a correção ou novo recurso funciona conforme esperado.
  • Tanto os novos testes quanto os antigos estão passando localmente.

Copy link

vercel bot commented Nov 2, 2024

@rodrigoKulb is attempting to deploy a commit to the TabNews Team on Vercel.

A member of the Team first needs to authorize it.

@aprendendofelipe
Copy link
Collaborator

Fala Rodrigão, obrigado pelo PR! 💪

Acho que o custo para manter esse código atualizado e útil não vai compensar algum benefício que ele traga.

E normalmente as mensagens de erro originais recebidas ao tentar rodar o projeto sem as dependências já são suficientes para entender qual é o problema.

O que acham?

@rodrigoKulb
Copy link
Contributor Author

Bom dia @aprendendofelipe, gosto da ideia de o "instalador" configurar automaticamente todas as dependências, mas sabemos que o Docker funciona de forma muito diferente em cada sistema operacional. Atualmente, apenas o Docker não está na lista de versões e instalação no package.json. Pesquisei alguma maneira de unificar isso no package.json, mas ele não faz parte dos pacotes do Node, então acabou ficando de fora.

Quanto à instalação do código, você se refere o número das duas versões? O restante é apenas if, exec e console.

De qualquer forma, foi só uma sugestão. Se você achar que o custo-benefício não compensa, seguimos sem! 👊️

Valeu pelo exercício! Aliás, você chegou a testar no Windows? Funcionou bem? Só testei no Linux. Só curiosidade mesmo!

…docker and compose dependency

script to check the installed docker and compose dependency

fix(correction of the last commit): i updated the docker version for testing to 30

I updated the docker version for testing to 30, generating the error in the test commit.
@rodrigoKulb rodrigoKulb force-pushed the checkDockerDependency branch from df540d0 to e74dce2 Compare November 8, 2024 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants