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

Quality of Life: Makefile #43

Closed
wants to merge 3 commits into from
Closed

Conversation

ppastorf
Copy link

@ppastorf ppastorf commented May 10, 2024

Este PR adiciona um Makefile, com algumas diretivas para desenvolvimento, fazendo referencia aos scripts do node configurados no package.json, e para serem futuramente usadas nas workflows de deploy:

  • Adicão do Makefile
  • Atualização do README.md
  • Atualização do workflow de deploy para usar

Por favor, confiram as diretivas criadas no Makefile, e opinem se tem algo errado / faltando

@ppastorf ppastorf requested a review from rafaelss May 11, 2024 15:55
@vanflux
Copy link

vanflux commented May 13, 2024

Parece só um wrapper dos scripts que já estão configurados no npm, não vejo vantagem visto que além de node e docker tem que ter mais o make instalado.

@CesarOliveira
Copy link

gosto do Make file, ajuda bastante novos devs a startar no projeto, geralmente uso em todos projetos que trabalho profissionalmente, vou testar a branch e se tiver ok deixo meu approve ✅

Copy link

@CesarOliveira CesarOliveira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ppastorf ppastorf mentioned this pull request May 15, 2024
@ppastorf
Copy link
Author

Parece só um wrapper dos scripts que já estão configurados no npm, não vejo vantagem visto que além de node e docker tem que ter mais o make instalado.

algumas diretivas vao ser usadas na a pipeline de deploy para facilitar

@vanflux
Copy link

vanflux commented May 17, 2024

Parece só um wrapper dos scripts que já estão configurados no npm, não vejo vantagem visto que além de node e docker tem que ter mais o make instalado.

algumas diretivas vao ser usadas na a pipeline de deploy para facilitar

Na minha visão só traz complexidade pro flow das coisas. Quando der problema, ta, mas oq o script sla oq do makefile faz? ah ele tem tal dependencia, vamos ver lá oq é... ah mais isso, mais aquilo. Mas posso estar errado.

@ppastorf ppastorf deleted the branch SOS-RS:develop May 18, 2024 23:04
@ppastorf ppastorf closed this May 18, 2024
@ppastorf ppastorf deleted the develop branch May 18, 2024 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants